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

Dubbo 监控统计数据有误 #274

Closed
Oliverwqcwrw opened this issue Jul 7, 2022 · 7 comments
Closed

Dubbo 监控统计数据有误 #274

Oliverwqcwrw opened this issue Jul 7, 2022 · 7 comments

Comments

@Oliverwqcwrw
Copy link

Question Report

在开始报告问题之前,请确保认真查看了以下步骤:

请在提交问题之前回答这些问题,谢谢。

你使用了哪个项目?Hippo-4J Server 还是 Hippo-4J Core?

Hippo-4J Core

你使用了哪个版本?

1.3.0

预期行为

可以监控dubbo线程池的变化并且动态修改,帮忙提供一下完整的yaml配置,如果需要配置Bean,也帮忙提供一下,非常感谢

实际行为

无法配置并监控dubbo线程池

原因分析(如果可以)

不知道如何配置,看文档只有一小部分

问题重现步骤

用于重现此问题的示例代码(例如 GitHub 链接)

@Oliverwqcwrw
Copy link
Author

Oliverwqcwrw commented Jul 7, 2022

我按官网的试了一下:
我的配置:

adapter-executors:
# threadPoolKey 代表线程池标识
- threadPoolKey: 'dubbo-provider'
# mark 为三方线程池框架类型,参见文初已支持框架集合
mark: 'Dubbo'
corePoolSize: 50
maximumPoolSize: 50

@bean
@DynamicThreadPool
public ThreadPoolExecutor dynamicThreadPoolExecutor() {
String threadPoolId = "dubbo-provider";
ThreadPoolExecutor dynamicExecutor = ThreadPoolBuilder.builder()
.threadFactory(threadPoolId)
.threadPoolId(threadPoolId)
.dynamicPool()
.build();
return dynamicExecutor;
}

prometheus拿到的数据是40和60:

image

@Oliverwqcwrw
Copy link
Author

Oliverwqcwrw commented Jul 8, 2022

通过调试,dubbo的线程数确实变了,应该是metric一直没有拿到dubbo的线程池的真实数据,一直拿的是默认的数据,想咨询一下dubbo的线程池监控怎么配置ThreadPoolExecutor呢?如果不配置没法监控,按照上面配置Bean拿不到真实的dubbo线程池的真实数据

@Oliverwqcwrw Oliverwqcwrw changed the title 你好,可以提供一个完整的dubbo框架线程池适配的配置吗?非常感谢 Dubbo 监控统计数据有误 Jul 10, 2022
@magestacks
Copy link
Member

这种测试方式有问题。adapter-executors 下面的配置属于 dubbo 框架原生线程池,而不是你通过代码自定义的线程池。

@Oliverwqcwrw
Copy link
Author

如果想要监控dubbo的线程池,需要怎么配置呢,我一开始只配置yaml没有配置Bean是不会有监控数据的,但是这种方式配置Bean应该也不太对,应该是针对自定义线程池才会这样配置,实在是没有其他办法了,然后我这样配置了一下Bean,修改了一下源码,然后可以做到监控dubbo线程池数据,如果目前对于这种引入hippo4j-core支持监控dubbo的话,辛苦发一下配置,非常感谢 :)

@magestacks
Copy link
Member

目前并不支持三方线程池监控功能。1.3.3 版本计划会把监控单独抽出去一个模块,以微内核的方式运行。

@Oliverwqcwrw
Copy link
Author

好吧,因为我们要做三方线程池监控并动态修改线程数,我会先通过自己修改源码的方式实现,期待支持三方线程池监控功能 :)

@Oliverwqcwrw
Copy link
Author

嗯那,还在调研阶段,如果采用了您的方案,我会登记的 您的设计确实很棒 :)

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

Successfully merging a pull request may close this issue.

2 participants