Jiashun Zhu

Results 33 comments of Jiashun Zhu

这个是listen fd被创建的时候出错了,看看这个端口状态是不是在正常没有被使用的状态,或者换个listen port

@DafanziQ 加入timer以后,线程池会在某些情况下crash,所以暂时先注释了。

@Tuvie 可以简单介绍一下代码结构和改动思路吗?文件有点多,有个整体的overview的话review起来会方便一些,谢谢。

> > @Tuvie 可以简单介绍一下代码结构和改动思路吗?文件有点多,有个整体的overview的话review起来会方便一些,谢谢。 > > 已在docs中添加相应描述 感谢。目前正在review,文件有点多,看完需要些时间。

LGTM. @Tuvie 从上次提交到现在有什么bugfix吗?没有的话等 @wwbmmm 看看线上运行的稳定程度,没什么问题的话可以考虑合入了。

@Tuvie 有conflicts,需要merge master重新提交一下

看起来是linker没有找到gflags,用cmake的话,看下CMakeCache.txt里面GFLAGS_LIBRARY=是不是正确找到了。如果都设置了,试一下mkdir build; cmake ..; make VERBOSE=1; 在链接的时候看下gflags有没有被加上去

Hi,能简单描述下这个PR的背景吗,是别的lib也依赖了gflags.so?

目前brpc内部统计的metric用gauge和summary就足够了,所以只输出了gauge和summary。如果需要用label来输出业务的统计数据,建议用Prometheus官方提供的cpp client