Kezhu Wang
Kezhu Wang
@fpj @sijie @ivankelly Could you please take time to evaluate this ?
第三段的翻译错乱了。 代码是原文的第三段,结论是原文的第四段。 幸亏点进原文看了下,要不然以为 JavaScript 又犯傻了。
@maoling @symat Could you please take a look at this ? It is valuable in testing/embedded environments to avoid port conflict. It is ZooKeeper's [CURATOR-535](https://issues.apache.org/jira/browse/CURATOR-535).
@eolivelli @symat Shall we merge this ?
@eolivelli @hanm @nkalmar @afine @fpj Could you please take a look at this pr ? I think it is related to #157 and #1445.
This fixes failed ci after #1821 merged(https://github.com/apache/zookeeper/commit/c05ca0ee0deedd50d0ef32e3e01ad7f691d27009) https://github.com/apache/zookeeper/runs/6568872986?check_suite_focus=true P.S. Place `Thread.sleep(200)` after `submittedRequests.take()` in `RequestThrottler#run` will fail these assertions also before #1821, but for different reason due to changes in...
@maoling @eolivelli @symat PTAL
I overlooked test failures unrelated to `testLargeRequestThrottling` in #1821. I try to fix all remaining test failures in `RequestThrottlerTest` in this pr. All of these failures could be reproduced by...
@maoling @symat Thank you for merging and reviewing.
Failed ci: * https://github.com/apache/zookeeper/runs/6982815844?check_suite_focus=true * https://github.com/kezhuw/zookeeper/runs/6996423868?check_suite_focus=true