[jira] [Updated] (ZOOKEEPER-706) large numbers of watches can cause session re-establishment to fail

2018-10-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-706?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-706: - Labels: pull-request-available (was: ) > large numbers of watches can cause session r

[GitHub] zookeeper pull request #683: MOD: Fix ZOOKEEPER-706 in c client when a lot w...

2018-10-30 Thread xfm18901105
GitHub user xfm18901105 opened a pull request: https://github.com/apache/zookeeper/pull/683 MOD: Fix ZOOKEEPER-706 in c client when a lot watchers reconnect the … …server You can merge this pull request into a Git repository by running: $ git pull https://github.com/xfm1890

[GitHub] zookeeper issue #678: ZOOKEEPER-3173: Quorum TLS - support PEM trust/key sto...

2018-10-30 Thread ivmaykov
Github user ivmaykov commented on the issue: https://github.com/apache/zookeeper/pull/678 Refactored common code shared by a bunch of tests that use `X509TestContext` into a new base class, `BaseX509ParameterizedTestCase`. ---

[GitHub] zookeeper issue #678: ZOOKEEPER-3173: Quorum TLS - support PEM trust/key sto...

2018-10-30 Thread ivmaykov
Github user ivmaykov commented on the issue: https://github.com/apache/zookeeper/pull/678 @eolivelli capitalized "Airlift" in NOTICE.txt @anmolnar use `FileNameUtils.getExtension()` for file type detection everyone: fixed some copy-paste bugs in PEMFileLoaderTest and JKSFileLoa

[GitHub] zookeeper pull request #678: ZOOKEEPER-3173: Quorum TLS - support PEM trust/...

2018-10-30 Thread ivmaykov
Github user ivmaykov commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/678#discussion_r229524910 --- Diff: zookeeper-server/src/main/java/org/apache/zookeeper/util/PemReader.java --- @@ -0,0 +1,224 @@ +/* + * Licensed under the Apache License

[GitHub] zookeeper pull request #678: ZOOKEEPER-3173: Quorum TLS - support PEM trust/...

2018-10-30 Thread hanm
Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/678#discussion_r229523540 --- Diff: zookeeper-server/src/main/java/org/apache/zookeeper/util/PemReader.java --- @@ -0,0 +1,224 @@ +/* + * Licensed under the Apache License, Ve

[GitHub] zookeeper issue #247: ZOOKEEPER-2778: Potential server deadlock between foll...

2018-10-30 Thread hanm
Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/247 Thanks guys for the reminder. I am getting back to it this week. ---

[GitHub] zookeeper issue #678: ZOOKEEPER-3173: Quorum TLS - support PEM trust/key sto...

2018-10-30 Thread ivmaykov
Github user ivmaykov commented on the issue: https://github.com/apache/zookeeper/pull/678 @anmolnar it's certainly easier to keep track of the copied code if we don't modify it much. Moving the logic into PEMFileLoader and making it non-static would make it harder to trace the history

ZooKeeper_branch35_jdk8 - Build # 1174 - Still Failing

2018-10-30 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch35_jdk8/1174/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 64.46 KB...] [junit] Running org.apache.zookeep

[GitHub] zookeeper issue #247: ZOOKEEPER-2778: Potential server deadlock between foll...

2018-10-30 Thread maoling
Github user maoling commented on the issue: https://github.com/apache/zookeeper/pull/247 - Sorry for my late.I will work on this issue at this weekend. - for other potential deadlock in the QuorumCnxManager and QuorumPeer,the [ZOOKEEPER-901](https://issues.apache.org/jira/brows

[jira] [Commented] (ZOOKEEPER-236) SSL Support for Atomic Broadcast protocol

2018-10-30 Thread Rakesh R (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16668331#comment-16668331 ] Rakesh R commented on ZOOKEEPER-236: Awesome! Great work everyone! [~andorm], a

[GitHub] zookeeper pull request #678: ZOOKEEPER-3173: Quorum TLS - support PEM trust/...

2018-10-30 Thread eolivelli
Github user eolivelli commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/678#discussion_r229202470 --- Diff: NOTICE.txt --- @@ -3,3 +3,9 @@ Copyright 2009-2014 The Apache Software Foundation This product includes software developed at T