[GitHub] zookeeper pull request #474: ZOOKEEPER-2977: Concurrency for addAuth corrupt...

2018-06-13 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/474#discussion_r195301371 --- Diff: src/java/test/org/apache/zookeeper/server/ServerCnxnTest.java --- @@ -0,0 +1,130 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] zookeeper pull request #474: ZOOKEEPER-2977: Concurrency for addAuth corrupt...

2018-06-13 Thread sumitagrawl
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] zookeeper pull request #474: ZOOKEEPER-2977: Concurrency for addAuth corrupt...

2018-06-13 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/474#discussion_r195182437 --- Diff: src/java/test/org/apache/zookeeper/server/ServerCnxnTest.java --- @@ -0,0 +1,130 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] zookeeper pull request #474: ZOOKEEPER-2977: Concurrency for addAuth corrupt...

2018-06-13 Thread sumitagrawl
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] zookeeper pull request #474: ZOOKEEPER-2977: Concurrency for addAuth corrupt...

2018-06-11 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/474#discussion_r194529403 --- Diff: src/java/test/org/apache/zookeeper/server/ServerCnxnTest.java --- @@ -0,0 +1,130 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] zookeeper pull request #474: ZOOKEEPER-2977: Concurrency for addAuth corrupt...

2018-03-08 Thread afine
Github user afine commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/474#discussion_r173243505 --- Diff: src/java/test/org/apache/zookeeper/server/ServerCnxnTest.java --- @@ -0,0 +1,129 @@ +/** + * Licensed to the Apache Software Foundation

[GitHub] zookeeper pull request #474: ZOOKEEPER-2977: Concurrency for addAuth corrupt...

2018-03-02 Thread afine
Github user afine commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/474#discussion_r171980950 --- Diff: src/java/test/org/apache/zookeeper/server/NIOServerCnxnTest.java --- @@ -103,4 +105,22 @@ public void testValidSelectionKey() throws Exception {

[GitHub] zookeeper pull request #474: ZOOKEEPER-2977: Concurrency for addAuth corrupt...

2018-02-26 Thread sumitagrawl
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 by