Failed: ZOOKEEPER- PreCommit Build #2257

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2257/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 81.10 MB...] [exec]

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

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

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user roodkcab commented on the issue: https://github.com/apache/zookeeper/pull/639 @hanm license header added for cert_creator.sh and ignore all others ---

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/639 @roodkcab we could exclude some of the files for `releaseaudit` target such as cert files (or even zoo.cfg) where adding Apache License header does not make any sense. See

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r220055378 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r220055291 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatcherCleaner.java --- @@ -0,0 +1,176 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r220050492 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r220050127 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r220049442 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r220049264 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatcherCleaner.java --- @@ -0,0 +1,176 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user roodkcab commented on the issue: https://github.com/apache/zookeeper/pull/639 seems Jenkins says No... I’ll have a check on java’s test cert files later ---

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user roodkcab commented on the issue: https://github.com/apache/zookeeper/pull/639 @hanm no problem, I’ll add Apache license headers to zoo.cfg and cert_creator.sh, .crt and .csr and .pkcs12 files are certificate files, which should not add license headers, May I add a

[GitHub] zookeeper issue #611: ZOOKEEPER-3131

Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/611 @anmolnar here are some clarifications based on your comments: * #611 is not needed anymore because the issue @wangchaod reported here is due to the 'leaking' watcher entry in

ZooKeeper-trunk - Build # 204 - Still Failing

See https://builds.apache.org/job/ZooKeeper-trunk/204/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 147.10 KB...] [junit] Running

[jira] [Commented] (ZOOKEEPER-2847) Cannot bind to client port when reconfig based on old static config

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16626539#comment-16626539 ] Michael Han commented on ZOOKEEPER-2847: [~yisong-yue] Thanks for quick response. bq. by

[jira] [Commented] (ZOOKEEPER-2847) Cannot bind to client port when reconfig based on old static config

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16626522#comment-16626522 ] Yisong Yue commented on ZOOKEEPER-2847: --- [~hanm] Yes, I'll put up a patch (by that you mean

Failed: ZOOKEEPER- PreCommit Build #2256

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2256/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 75.78 MB...] [exec]

[GitHub] zookeeper issue #633: ZOOKEEPER-3032 - MAVEN MIGRATION - move java server, c...

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

[jira] [Assigned] (ZOOKEEPER-925) Consider maven site generation to replace our forrest site and documentation generation

[ https://issues.apache.org/jira/browse/ZOOKEEPER-925?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tamas Penzes reassigned ZOOKEEPER-925: -- Assignee: Tamas Penzes (was: Patrick Hunt) > Consider maven site generation to

Failed: ZOOKEEPER- PreCommit Build #2255

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2255/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 67.74 MB...] [exec]

[GitHub] zookeeper issue #642: ZOOKEEPER-3151: test Jenkins. Don't merge.

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

Failed: ZOOKEEPER- PreCommit Build #2254

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2254/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 81.49 MB...] [exec]

[GitHub] zookeeper issue #633: ZOOKEEPER-3032 - MAVEN MIGRATION - move java server, c...

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

[GitHub] zookeeper issue #642: ZOOKEEPER-3151: test Jenkins. Don't merge.

Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/642 test this please ---

[GitHub] zookeeper issue #642: ZOOKEEPER-3151: test Jenkins. Don't merge.

Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/642 test ---

[GitHub] zookeeper issue #642: ZOOKEEPER-3151: test Jenkins. Don't merge.

Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/642 please ---

[jira] [Commented] (ZOOKEEPER-2847) Cannot bind to client port when reconfig based on old static config

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16626486#comment-16626486 ] Michael Han commented on ZOOKEEPER-2847: [~yisong-yue] Do you want to put up another patch to

[jira] [Reopened] (ZOOKEEPER-2847) Cannot bind to client port when reconfig based on old static config

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Han reopened ZOOKEEPER-2847: > Cannot bind to client port when reconfig based on old static config >

[jira] [Commented] (ZOOKEEPER-2847) Cannot bind to client port when reconfig based on old static config

[ https://issues.apache.org/jira/browse/ZOOKEEPER-2847?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16626485#comment-16626485 ] Michael Han commented on ZOOKEEPER-2847: looks like this patch breaks recent trunk build.

ZooKeeper-trunk - Build # 203 - Still Failing

See https://builds.apache.org/job/ZooKeeper-trunk/203/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 147.18 KB...] [junit] Tests run: 1, Failures: 0,

Failed: ZOOKEEPER- PreCommit Build #2253

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2253/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 79.52 MB...] [exec]

[GitHub] zookeeper issue #642: ZOOKEEPER-3151: test Jenkins. Don't merge.

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

[GitHub] zookeeper issue #590: [ZOOKEEPER-1177] Add the memory optimized watch manage...

Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/590 thanks @lvfangmin for detailed reply. just had another review pass over some files i forgot to look last time. overall looks good. will sign this off once all review comments are addressed.

[GitHub] zookeeper pull request #642: ZOOKEEPER-3151: test Jenkins. Don't merge.

GitHub user hanm opened a pull request: https://github.com/apache/zookeeper/pull/642 ZOOKEEPER-3151: test Jenkins. Don't merge. You can merge this pull request into a Git repository by running: $ git pull https://github.com/hanm/zookeeper ZOOKEEPER-3151 Alternatively you can

[GitHub] zookeeper issue #633: ZOOKEEPER-3032 - MAVEN MIGRATION - move java server, c...

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

Failed: ZOOKEEPER- PreCommit Build #2252

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2252/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 68.37 MB...] [exec]

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219958886 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219912160 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219931441 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219911008 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatcherCleaner.java --- @@ -0,0 +1,176 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219957465 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219934138 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219912709 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache Software

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219959421 --- Diff: src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java --- @@ -0,0 +1,355 @@ +/** + * Licensed to the Apache Software

ZooKeeper-trunk-windows-cmake - Build # 2949 - Still Failing

See https://builds.apache.org/job/ZooKeeper-trunk-windows-cmake/2949/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 9.79 KB...] [ivy:retrieve] confs: [javacc]

Failed: ZOOKEEPER- PreCommit Build #2251

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2251/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 67.38 MB...] [exec]

[GitHub] zookeeper issue #633: ZOOKEEPER-3032 - MAVEN MIGRATION - move java server, c...

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

Failed: ZOOKEEPER- PreCommit Build #2250

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2250/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 68.19 MB...] [exec]

[GitHub] zookeeper issue #633: ZOOKEEPER-3032 - MAVEN MIGRATION - move java server, c...

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

Failed: ZOOKEEPER- PreCommit Build #2249

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2249/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 74.73 MB...] [exec]

[GitHub] zookeeper issue #633: ZOOKEEPER-3032 - MAVEN MIGRATION - move java server, c...

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

Re: New precommit job 'retest this please' trick

Awesome! On 09/24/2018 06:27 PM, Michael Han wrote: >>> How the new process any different from that? > Jenkins job triggered by "Rebuild" button can't write back the job result > to github, leading to a yellow pending check status even after the job > finished; while "test this please" does not

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

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

Failed: ZOOKEEPER- PreCommit Build #2248

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2248/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 80.80 MB...] [exec] Test

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/639 @roodkcab Some new files are missing apache license header, which leads to release audit failure:

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/639 @eolivelli maybe, but it works. i think the trigger phrase was looking for a substring match only. ---

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

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

Failed: ZOOKEEPER- PreCommit Build #2247

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2247/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 82.16 MB...] [exec] Test

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user eolivelli commented on the issue: https://github.com/apache/zookeeper/pull/639 @hanm your phrase was wrong ---

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user eolivelli commented on the issue: https://github.com/apache/zookeeper/pull/639 retest this please ---

Failed: ZOOKEEPER- PreCommit Build #2246

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2246/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 73.56 MB...] [exec] +1

Re: New precommit job 'retest this please' trick

>> How the new process any different from that? Jenkins job triggered by "Rebuild" button can't write back the job result to github, leading to a yellow pending check status even after the job finished; while "test this please" does not have this issue. On Mon, Sep 24, 2018 at 7:01 AM, Andor

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/639 test this please ---

[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/590#discussion_r219896267 --- Diff: src/java/main/org/apache/zookeeper/server/watch/BitHashSet.java --- @@ -0,0 +1,156 @@ +/** + * Licensed to the Apache Software Foundation

Re: New precommit job 'retest this please' trick

I usually logged in to Jenkins for other reasons, but that's also fine to have this option. Cheers Andor On Mon, Sep 24, 2018 at 3:51 PM, Enrico Olivelli wrote: > Il lun 24 set 2018, 15:25 Andor Molnar ha > scritto: > > > Thanks Enrico. Afaik the Rebuild button was already available for > >

Re: New precommit job 'retest this please' trick

Il lun 24 set 2018, 15:25 Andor Molnar ha scritto: > Thanks Enrico. Afaik the Rebuild button was already available for > committers. > How the new process any different from th > In order to relaunch tests you do not need to go to jenkins but simply add a 'retest this please' comment on the PR

Re: New precommit job 'retest this please' trick

Thanks Enrico. Afaik the Rebuild button was already available for committers. How the new process any different from that? Andor On Sun, Sep 23, 2018 at 8:09 AM, Enrico Olivelli wrote: > Hi ZooKeepers, > we have enhanced the "precommit" automated QA testing. > Before merging any patch the

Build failed in Jenkins: ZooKeeper-trunk-owasp #120

See -- [...truncated 21.04 KB...] [javac] T extends Object declared in class Class [javac]

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

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

Failed: ZOOKEEPER- PreCommit Build #2245

Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2245/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 82.21 MB...] [exec] Test

[GitHub] zookeeper issue #639: ZOOKEEPER-2122: add openssl to zookeeper c client to s...

Github user roodkcab commented on the issue: https://github.com/apache/zookeeper/pull/639  @hanm unit test for ssl added. ---