Lei Zhiyuan

Results 122 comments of Lei Zhiyuan
trafficstars

> Thanks for your contribution > > Can you explain more about the motivation for this change? Thanks for your reply , now, we have different bk clusters, if all...

After I checked the testcase , I find it seems disableHttpDebugMethods do the same thing. https://github.com/apache/pulsar/pull/7907 we can set disableHttpDebugMethods=true to solve the issue? and I can add the disableHttpDebugMethods...

> > After I checked the testcase , I find it seems disableHttpDebugMethods do the same thing. #7907 > > we can set disableHttpDebugMethods=true to solve the issue? > >...

> @leizhiyuan I think we don't need to close the issue for now. Maybe you want to push a fix for it? I think I can assign to you directly....

maybe we should use GetStatsOptions next pr, not original name like getPreciseBacklog,subscriptionBacklogSize etc.

> There is a failed broker test in your [leizhiyuan#4](https://github.com/leizhiyuan/pulsar/pull/4) : https://github.com/leizhiyuan/pulsar/actions/runs/3373856231/jobs/5598944551#step:10:1209 > > if it is caused by your change please fix it. https://github.com/leizhiyuan/pulsar/pull/5 I didn't sync the master...

> > There is a failed broker test in your [leizhiyuan#4](https://github.com/leizhiyuan/pulsar/pull/4) : https://github.com/leizhiyuan/pulsar/actions/runs/3373856231/jobs/5598944551#step:10:1209 > > if it is caused by your change please fix it. > > [leizhiyuan#5](https://github.com/leizhiyuan/pulsar/pull/5) > >...

辛苦解决一下冲突。

以前代码是基于jdk6的。一直兼容jdk6.所以这个也没用,我们看一下