-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pika-exporter:Add RocksDB column family metrics(#1566) #1696
Conversation
I think we should discuss which indicators are more important, and expose indicators that are more conducive to helping O&M personnel monitor Pika on grafana. For example, io_stalls can reflect disk I/O performance very well, but there are many compactions stats, so you need to consider whether to display or display part. |
I discussed your suggestion with my DBA colleagues. Perhaps due to the lack of annotation information and experience in use, it is difficult for everyone to judge the specific practical significance of such indicators. This week, I will try to sort out the specific meaning of each indicator to facilitate discussion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A modification suggestion is mainly to supplement the description information of the indicator.
I think this PR can be merged after modification. I discussed this with @AlexStocks. After the plan, you can start a new PR, the pika_exporter
obtains the method of Metrics into the form of configurable to obtain a part of the information. Now obtained through the info all
command. After, then it can be disassembled into multiple info XXX
commands to obtain metrics.
What do you think of this plan?
@chenbt-hz need to rebase |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…) (OpenAtomFoundation#1696) * 支持cfstats监控数据采集 * 修改监控指标注释;修复时间解析错误bug;添加mac-amd64编译 * 添加监控指标说明 * bug fix
…) (OpenAtomFoundation#1696) * 支持cfstats监控数据采集 * 修改监控指标注释;修复时间解析错误bug;添加mac-amd64编译 * 添加监控指标说明 * bug fix
Fixes: #1566
新增以下的监控指标:
已zset为例,以下是info all的结果
采集的数据大致如下: