[GitHub] zookeeper issue #666: ZOOKEEPER-1908: setAcl should be have a recursive func...

2018-10-09 Thread Reidddddd
Github user Reidd commented on the issue: https://github.com/apache/zookeeper/pull/666 I checkouted a new branch based on branch-3.5, and the PR base branch i chose is branch-3.5. It should not have conflicts, anywhere i missed? ping @anmolnar ---

[GitHub] zookeeper issue #666: ZOOKEEPER-1908: setAcl should be have a recursive func...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/666 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2403/ ---

Failed: ZOOKEEPER- PreCommit Build #2403

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2403/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 6.20 KB...] SSH_CONNECTION=195.201.2

Failed: ZOOKEEPER- PreCommit Build #2402

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2402/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 5.95 KB...] at hudson.model.

[GitHub] zookeeper issue #666: ZOOKEEPER-1908: setAcl should be have a recursive func...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/666 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2402/ ---

[GitHub] zookeeper pull request #666: ZOOKEEPER-1908: setAcl should be have a recursi...

2018-10-09 Thread Reidddddd
GitHub user Reidd opened a pull request: https://github.com/apache/zookeeper/pull/666 ZOOKEEPER-1908: setAcl should be have a recursive function Backport to branch-3.5. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Reidd

ZooKeeper-trunk - Build # 225 - Still Failing

2018-10-09 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper-trunk/225/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 195.78 KB...] [junit] Running org.apache.zookeeper.serve

[GitHub] zookeeper issue #628: ZOOKEEPER-3140: Allow Followers to host Observers

2018-10-09 Thread enixon
Github user enixon commented on the issue: https://github.com/apache/zookeeper/pull/628 Any suggestions on the C zktest-mt failures? This patch does not touch the c client or the default configurations for those tests so I'm unsure how to proceed. ---

[GitHub] zookeeper pull request #628: ZOOKEEPER-3140: Allow Followers to host Observe...

2018-10-09 Thread enixon
Github user enixon commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/628#discussion_r223889253 --- Diff: src/java/test/org/apache/zookeeper/server/util/PortForwarder.java --- @@ -29,62 +26,15 @@ import java.net.Socket; import java.net.Socke

[GitHub] zookeeper pull request #628: ZOOKEEPER-3140: Allow Followers to host Observe...

2018-10-09 Thread enixon
Github user enixon commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/628#discussion_r223888479 --- Diff: src/java/main/org/apache/zookeeper/server/quorum/Learner.java --- @@ -84,8 +84,12 @@ public Socket getSocket() { protected stat

[GitHub] zookeeper issue #628: ZOOKEEPER-3140: Allow Followers to host Observers

2018-10-09 Thread enixon
Github user enixon commented on the issue: https://github.com/apache/zookeeper/pull/628 Going to fix this patch up by rebasing and reworking the base diff to operate against zookeeper-server/src/ and zookeeper-common/src/test instead of the old paradigm src/java. ---

[GitHub] zookeeper issue #663: ZOOKEEPER-3154: Update release process to use the Mark...

2018-10-09 Thread tamaashu
Github user tamaashu commented on the issue: https://github.com/apache/zookeeper/pull/663 Thanks @lvfangmin and @eolivelli to pointing this out. 2.6.0 was a typo in the master branch (should have been 3.6.0, will fix it in master tomorrow) and got backported here accidentally. 3.5

[GitHub] zookeeper pull request #663: ZOOKEEPER-3154: Update release process to use t...

2018-10-09 Thread eolivelli
Github user eolivelli commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/663#discussion_r223854802 --- Diff: pom.xml --- @@ -30,7 +30,7 @@ org.apache.zookeeper zookeeper pom - 2.6.0-SNAPSHOT + 3.5.5-beta-SNAPSHOT ---

[GitHub] zookeeper issue #624: ZOOKEEPER-3108:use a new property server.id in the zoo...

2018-10-09 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/624 Previously, it was not part of the dynamic config, because it’s in a separate config file. But after this change it will move to the dynamic config, if I remember correctly it check and move a

[GitHub] zookeeper pull request #663: ZOOKEEPER-3154: Update release process to use t...

2018-10-09 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/663#discussion_r223838835 --- Diff: pom.xml --- @@ -30,7 +30,7 @@ org.apache.zookeeper zookeeper pom - 2.6.0-SNAPSHOT + 3.5.5-beta-SNAPSHOT ---

[GitHub] zookeeper issue #665: [ZOOKEEPER-3163] Use session map in the Netty to impro...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/665 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2401/ ---

Failed: ZOOKEEPER- PreCommit Build #2401

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2401/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 82.70 MB...] [exec] ==

[jira] [Updated] (ZOOKEEPER-3163) Use session map to improve the performance when closing session in Netty

2018-10-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-3163: -- Labels: pull-request-available (was: ) > Use session map to improve the performanc

[GitHub] zookeeper pull request #665: [ZOOKEEPER-3163] Use session map in the Netty t...

2018-10-09 Thread lvfangmin
GitHub user lvfangmin opened a pull request: https://github.com/apache/zookeeper/pull/665 [ZOOKEEPER-3163] Use session map in the Netty to improve close session performance This is a refactor to make the Netty able to use the same closeSession logic in NIOServerCnxn, which is more

Failed: ZOOKEEPER- PreCommit Build #2400

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2400/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 8.22 KB...] [exec] ===

[GitHub] zookeeper issue #664: ZOOKEEPER-3154: Update release process to use the Mark...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/664 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2400/ ---

[GitHub] zookeeper pull request #664: ZOOKEEPER-3154: Update release process to use t...

2018-10-09 Thread tamaashu
GitHub user tamaashu opened a pull request: https://github.com/apache/zookeeper/pull/664 ZOOKEEPER-3154: Update release process to use the MarkDown solution Added `mvn clean` and `mvn install` to `ant clean` and `ant compile` as dependency. This way we clean and build zookeeper-

Failed: ZOOKEEPER- PreCommit Build #2397

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2397/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 81.04 MB...] [exec] [exec

[GitHub] zookeeper issue #662: ZOOKEEPER-3162. Broken lock semantics in C client lock...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/662 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2397/ ---

Failed: ZOOKEEPER- PreCommit Build #2399

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2399/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 8.40 KB...] [exec] ===

[GitHub] zookeeper issue #661: ZOOKEEPER-3154: Update release process to use the Mark...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/661 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2399/ ---

[GitHub] zookeeper issue #661: ZOOKEEPER-3154: Update release process to use the Mark...

2018-10-09 Thread tamaashu
Github user tamaashu commented on the issue: https://github.com/apache/zookeeper/pull/661 retest this please ---

[GitHub] zookeeper issue #658: ZOOKEEPER-3032 - MAVEN MIGRATION - branch 3.4 move jav...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/658 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2395/ ---

Success: ZOOKEEPER- PreCommit Build #2395

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2395/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 41.31 MB...] [exec] +1 core

[GitHub] zookeeper issue #661: ZOOKEEPER-3154: Update release process to use the Mark...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/661 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2398/ ---

Failed: ZOOKEEPER- PreCommit Build #2398

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2398/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 8.32 KB...] [exec] ===

[GitHub] zookeeper issue #661: ZOOKEEPER-3154: Update release process to use the Mark...

2018-10-09 Thread tamaashu
Github user tamaashu commented on the issue: https://github.com/apache/zookeeper/pull/661 retest this please ---

[GitHub] zookeeper issue #662: ZOOKEEPER-3162. Broken lock semantics in C client lock...

2018-10-09 Thread andreareale
Github user andreareale commented on the issue: https://github.com/apache/zookeeper/pull/662 retest this please ---

Failed: ZOOKEEPER- PreCommit Build #2396

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2396/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 7.74 KB...] [exec] [exec]

[GitHub] zookeeper issue #663: ZOOKEEPER-3154: Update release process to use the Mark...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/663 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2396/ ---

[GitHub] zookeeper pull request #663: ZOOKEEPER-3154: Update release process to use t...

2018-10-09 Thread tamaashu
GitHub user tamaashu opened a pull request: https://github.com/apache/zookeeper/pull/663 ZOOKEEPER-3154: Update release process to use the MarkDown solution Added `mvn clean` and `mvn install` to `ant clean` and `ant compile` as dependency. This way we clean and build zookeeper-

[GitHub] zookeeper issue #658: ZOOKEEPER-3032 - MAVEN MIGRATION - branch 3.4 move jav...

2018-10-09 Thread nkalmar
Github user nkalmar commented on the issue: https://github.com/apache/zookeeper/pull/658 retest this please ---

[GitHub] zookeeper issue #662: ZOOKEEPER-3162. Broken lock semantics in C client lock...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/662 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2394/ ---

Failed: ZOOKEEPER- PreCommit Build #2394

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2394/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 76.26 MB...] [exec] [exec

[jira] [Updated] (ZOOKEEPER-3163) Use session map to improve the performance when closing session in Netty

2018-10-09 Thread Fangmin Lv (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fangmin Lv updated ZOOKEEPER-3163: -- Description: Previously, it needs to go through all the cnxns to find out the session to c

[jira] [Updated] (ZOOKEEPER-3163) Use session map to improve the performance when closing session in Netty

2018-10-09 Thread Fangmin Lv (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Fangmin Lv updated ZOOKEEPER-3163: -- Description: Previously, it needs to go through all the cnxns to find out the session to c

[jira] [Created] (ZOOKEEPER-3163) Use session map to improve the performance when closing session in Netty

2018-10-09 Thread Fangmin Lv (JIRA)
Fangmin Lv created ZOOKEEPER-3163: - Summary: Use session map to improve the performance when closing session in Netty Key: ZOOKEEPER-3163 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3163 Proj

[jira] [Updated] (ZOOKEEPER-3162) Broken lock semantics in C client lock-recipe

2018-10-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3162?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-3162: -- Labels: pull-request-available (was: ) > Broken lock semantics in C client lock-re

[GitHub] zookeeper pull request #662: ZOOKEEPER-3162

2018-10-09 Thread andreareale
GitHub user andreareale opened a pull request: https://github.com/apache/zookeeper/pull/662 ZOOKEEPER-3162 This PR fixes a few issues with the C client lock-recipe, as documented in more detailed in [ZOOKEEPER-3162](https://issues.apache.org/jira/browse/ZOOKEEPER-3162) on JIRA.

Failed: ZOOKEEPER- PreCommit Build #2393

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2393/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 7.68 KB...] [exec] Testing

[GitHub] zookeeper issue #661: ZOOKEEPER-3154: Update release process to use the Mark...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/661 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2393/ ---

[jira] [Updated] (ZOOKEEPER-3154) Update release process to use the MarkDown solution

2018-10-09 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3154?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-3154: -- Labels: pull-request-available (was: ) > Update release process to use the MarkDow

[GitHub] zookeeper pull request #661: ZOOKEEPER-3154: Update release process to use t...

2018-10-09 Thread tamaashu
GitHub user tamaashu opened a pull request: https://github.com/apache/zookeeper/pull/661 ZOOKEEPER-3154: Update release process to use the MarkDown solution Added `mvn clean` and `mvn install` to `ant clean` and `ant compile` as dependency. This way we clean and build zookeeper-

[jira] [Created] (ZOOKEEPER-3162) Broken lock semantics in C client lock-recipe

2018-10-09 Thread Andrea Reale (JIRA)
Andrea Reale created ZOOKEEPER-3162: --- Summary: Broken lock semantics in C client lock-recipe Key: ZOOKEEPER-3162 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3162 Project: ZooKeeper

[GitHub] zookeeper issue #658: ZOOKEEPER-3032 - MAVEN MIGRATION - branch 3.4 move jav...

2018-10-09 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/658 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2392/ ---

Failed: ZOOKEEPER- PreCommit Build #2392

2018-10-09 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2392/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 39.09 MB...] [exec] [exec

ZooKeeper_branch35_jdk8 - Build # 1148 - Still Failing

2018-10-09 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch35_jdk8/1148/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 108.01 KB...] [junit] Tests run: 3, Failures: 0

[GitHub] zookeeper issue #587: ZOOKEEPER-3106: Zookeeper client supports IPv6 address...

2018-10-09 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/587 Thanks @maoling . Go ahead please! ---

[GitHub] zookeeper issue #660: ZOOKEEPER-2320. C-client crashes when removing watcher...

2018-10-09 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/660 @lvfangmin Assertion failied in the unit test `testRemoveWatchers`. Investigating. ---

Re: Maven migration - main src dir moved

2018-10-09 Thread Norbert Kalmar
Thanks Maoling for reporting this! After I finish the 3.4 migration as well (today, hopefully), I will get to testing thoroughly the existing ant build with the new directory structure on all 3 branches, and also fix the problems you reported. There's already a jira planned for this - ZOOKEEPER-31

[jira] [Updated] (ZOOKEEPER-3122) Verify ant build after directory migration and the end artifact

2018-10-09 Thread Norbert Kalmar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3122?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norbert Kalmar updated ZOOKEEPER-3122: -- Summary: Verify ant build after directory migration and the end artifact (was: Ver

[jira] [Updated] (ZOOKEEPER-3028) Placeholder Jira

2018-10-09 Thread Norbert Kalmar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3028?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norbert Kalmar updated ZOOKEEPER-3028: -- Description: This step is no longer valid, keeping under maven migration umbrella J

[jira] [Updated] (ZOOKEEPER-3028) Placeholder Jira

2018-10-09 Thread Norbert Kalmar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3028?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norbert Kalmar updated ZOOKEEPER-3028: -- Summary: Placeholder Jira (was: Step 3 - Separate java server and client code) >

[jira] [Commented] (ZOOKEEPER-1029) C client bug in zookeeper_init (if bad hostname is given)

2018-10-09 Thread Jorge Machado (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-1029?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16642943#comment-16642943 ] Jorge Machado commented on ZOOKEEPER-1029: -- Hi [~cnauroth] I think there is

[GitHub] zookeeper issue #658: ZOOKEEPER-3032 - MAVEN MIGRATION - branch 3.4 move jav...

2018-10-09 Thread nkalmar
Github user nkalmar commented on the issue: https://github.com/apache/zookeeper/pull/658 Sorry, I know this has some issue, but I escaped the digital world for the past 3 days. I will get back to this PR now... ---

[GitHub] zookeeper pull request #656: ZOOKEEPER-3032: MAVEN MIGRATION - branch-3.5 - ...

2018-10-09 Thread nkalmar
Github user nkalmar closed the pull request at: https://github.com/apache/zookeeper/pull/656 ---