Skip to content
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

nsq 全量指标插件 #649

Merged
merged 2 commits into from Sep 13, 2023
Merged

nsq 全量指标插件 #649

merged 2 commits into from Sep 13, 2023

Conversation

tianyanli
Copy link
Contributor

指标列表

nsq_client类

ready_count 可消费消息数
inflight_count 正在处理消息数
message_count 消息总数
finish_count 完成统计
requeue_count 重新排队消息数

nsq_channel类

depth 当前的积压量
backend_depth 消息缓冲队列积压量
inflight_count 正在处理消息数
deferred_count 延迟消息数
message_count 消息总数
requeue_count 重新排队消息数
timeout_count 超时消息数
client_count 客户端数量

nsq_topic类

depth 消息队列积压量
backend_depth 消息缓冲队列积压量
message_count 消息总数
channel_count 消费者总数

@kongfei605
Copy link
Collaborator

插件很赞 。 看了下逻辑,是把原来的插件挪到nsq_slim中了,是否可以把新逻辑添加到nsq 插件中?
现在的模式,用户需要去修改原来的配置目录名称。

如果能做成一个nsq 插件,然后兼容老的配置 就更好了。
比如用ins.Targets 表示多个url ,然后老的用户配置的ins.URL 可以先做url.Parse() ,去掉path后放到ins.Targets中,就走新采集逻辑就好了。这样,老的代码可以删除掉了。如果没时间,合并和兼容可以我来搞哈

@tianyanli
Copy link
Contributor Author

看了下逻辑,是把插件挪到了 nsq_slim 中了,是否可以把新逻辑添加到 nsq 插件中?现在的模式,用户需要去 原来的配置目录名称。

如果能支持一个nsq插件,然后兼容老的配置就更好了。 比如用ins.Targets表示多个url,然后老的用户配置的ins.URL可以先做url.Parse(),去掉path后放到ins.Targets中,就走新采集逻辑就好了。这样,老的代码就可以删掉了。如果没时间,合并和兼容可以我来搞哈

已实现新旧nsq插件配置兼容。

@kongfei605 kongfei605 merged commit e85f697 into flashcatcloud:main Sep 13, 2023
@kongfei605
Copy link
Collaborator

Thank you @tianyanli

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants