You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by rakeshadr <gi...@git.apache.org> on 2018/03/01 11:12:00 UTC
[GitHub] zookeeper issue #474: ZOOKEEPER-2977: Concurrency for addAuth corrupts quoru...
Github user rakeshadr commented on the issue:
https://github.com/apache/zookeeper/pull/474
@sumitagrawl Good analysis and fix. Few minor comments,
- Please take care indentation in the tests. Presently, it uses tab.
- Please add comments in test as a quick reference to the issue to show the expectations and background of this test, something like, https://github.com/apache/zookeeper/blob/master/src/java/test/org/apache/zookeeper/server/DataTreeTest.java#L70
- break; is not needed in loop, pls remove.
---