Github user sumitagrawl commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/474#discussion_r195275508
--- Diff: src/java/test/org/apache/zookeeper/server/ServerCnxnTest.java ---
@@ -0,0 +1,130 @@
+/**
+ * Licensed to the Apache Software
Github user sumitagrawl commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/474#discussion_r195170907
--- Diff: src/java/test/org/apache/zookeeper/server/ServerCnxnTest.java ---
@@ -0,0 +1,130 @@
+/**
+ * Licensed to the Apache Software
Github user sumitagrawl commented on the issue:
https://github.com/apache/zookeeper/pull/474
@maoling @afine Comments are fixed, please check
---
Github user sumitagrawl commented on the issue:
https://github.com/apache/zookeeper/pull/474
@rakeshadr @anmolnar @afine
Changes are done creating separate ServerCnxnTest.java for this purpose to
avoid test duplicates. Also all above review comments are handled.
---
GitHub user sumitagrawl opened a pull request:
https://github.com/apache/zookeeper/pull/474
ZOOKEEPER-2977: Concurrency for addAuth corrupts quorum packets
get copy of authInfo in ServerCnxn to avoid corrupt of quorum packets
You can merge this pull request into a Git repository