Re: About Jenkins pre-commit job

2018-09-19 Thread Enrico Olivelli
Il gio 20 set 2018, 01:17 Michael Han ha scritto: > We've seen lots of confusions recently around Jenkins pre-commit check > status on github pull request as well as questions like how to trigger > Jenkins job to get a green build. I added a section "Jenkins Pre-commit > Check" to our wiki page

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

2018-09-19 Thread hanm
Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/628#discussion_r219036460 --- Diff: src/java/main/org/apache/zookeeper/server/quorum/Follower.java --- @@ -196,6 +224,10 @@ protected long getLastQueued() { return

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

2018-09-19 Thread hanm
Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/628#discussion_r219034842 --- Diff: src/java/test/org/apache/zookeeper/server/quorum/DelayRequestProcessor.java --- @@ -0,0 +1,77 @@ +/** + * Licensed to the Apache Software

Re: Slack for communication

2018-09-19 Thread Michael Han
We have a gitter chat room: https://gitter.im/apache/apache-zookeeper. We also have https://zookeeper.apache.org/irc.html, which is less active these days. On Wed, Sep 19, 2018 at 8:09 PM, 岭秀 wrote: > Slack.Hi zk guys:Do we hava a slack for communication?then we can have > some talks except

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

2018-09-19 Thread hanm
Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/633 I think github web interface does not support `--follow` at the moment, unfortunately, but there is a chrome extension that seems working

Slack for communication

2018-09-19 Thread 岭秀
Slack.Hi zk guys:Do we hava a slack for communication?then we can have some talks except codes! Best regards maoling Beijing,China

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

2018-09-19 Thread nkalmar
Github user nkalmar commented on the issue: https://github.com/apache/zookeeper/pull/633 How can I do that? I tried a few things like git mv, but didn't work either. As far as I know, this is how git works unfortunately. But it did not loose the history, it's just that git is

ZooKeeper-trunk - Build # 197 - Still Failing

2018-09-19 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper-trunk/197/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 146.40 KB...] [junit] Tests run: 1, Failures: 0,

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

2018-09-19 Thread hanm
Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/633 Can we make sure to preserve the file commit history while moving these files around? I noticed recently after `src/c` was moved the entire commit history was lost. The commit history is a priceless

About Jenkins pre-commit job

2018-09-19 Thread Michael Han
We've seen lots of confusions recently around Jenkins pre-commit check status on github pull request as well as questions like how to trigger Jenkins job to get a green build. I added a section "Jenkins Pre-commit Check" to our wiki page about this matter:

[jira] [Resolved] (ZOOKEEPER-3151) Jenkins github integration is broken if retriggering the precommit job through Jenkins admin web page.

2018-09-19 Thread Michael Han (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3151?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael Han resolved ZOOKEEPER-3151. Resolution: Workaround Provided work around in

[GitHub] zookeeper pull request #634: ZOOKEEPER-3151: Test build bot 1. Don't merge.

2018-09-19 Thread hanm
Github user hanm closed the pull request at: https://github.com/apache/zookeeper/pull/634 ---

[GitHub] zookeeper pull request #635: ZOOKEEPER-3151: Test build bot 2. Don't merge.

2018-09-19 Thread hanm
Github user hanm closed the pull request at: https://github.com/apache/zookeeper/pull/635 ---

Success: ZOOKEEPER- PreCommit Build #2209

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

[jira] [Commented] (ZOOKEEPER-3151) Jenkins github integration is broken if retriggering the precommit job through Jenkins admin web page.

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16621203#comment-16621203 ] Hadoop QA commented on ZOOKEEPER-3151: -- +1 overall. GitHub Pull Request Build +1

[GitHub] zookeeper pull request #635: ZOOKEEPER-3151: Test build bot 2. Don't merge.

2018-09-19 Thread hanm
Github user hanm closed the pull request at: https://github.com/apache/zookeeper/pull/635 ---

[GitHub] zookeeper pull request #635: ZOOKEEPER-3151: Test build bot 2. Don't merge.

2018-09-19 Thread hanm
GitHub user hanm reopened a pull request: https://github.com/apache/zookeeper/pull/635 ZOOKEEPER-3151: Test build bot 2. Don't merge. You can merge this pull request into a Git repository by running: $ git pull https://github.com/hanm/zookeeper ZOOKEEPER-3151-2 Alternatively

[jira] [Commented] (ZOOKEEPER-3032) Step 1.6 - Create zk-server maven structure

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3032?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16621156#comment-16621156 ] Hadoop QA commented on ZOOKEEPER-3032: -- -1 overall. GitHub Pull Request Build +1

Failed: ZOOKEEPER- PreCommit Build #2208

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

[jira] [Commented] (ZOOKEEPER-3151) Jenkins github integration is broken if retriggering the precommit job through Jenkins admin web page.

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16621148#comment-16621148 ] Hadoop QA commented on ZOOKEEPER-3151: -- -1 overall. GitHub Pull Request Build +1

Failed: ZOOKEEPER- PreCommit Build #2205

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

[jira] [Commented] (ZOOKEEPER-3032) Step 1.6 - Create zk-server maven structure

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3032?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16621135#comment-16621135 ] Hadoop QA commented on ZOOKEEPER-3032: -- -1 overall. GitHub Pull Request Build +1

Failed: ZOOKEEPER- PreCommit Build #2207

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

Failed: ZOOKEEPER- PreCommit Build #2206

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

Failed: ZOOKEEPER- PreCommit Build #2204

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

[jira] [Commented] (ZOOKEEPER-3151) Jenkins github integration is broken if retriggering the precommit job through Jenkins admin web page.

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16621115#comment-16621115 ] Hadoop QA commented on ZOOKEEPER-3151: -- -1 overall. GitHub Pull Request Build +1

Failed: ZOOKEEPER- PreCommit Build #2202

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

[jira] [Commented] (ZOOKEEPER-3151) Jenkins github integration is broken if retriggering the precommit job through Jenkins admin web page.

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16621110#comment-16621110 ] Hadoop QA commented on ZOOKEEPER-3151: -- -1 overall. GitHub Pull Request Build +1

[GitHub] zookeeper pull request #635: ZOOKEEPER-3151: Test build bot 2. Don't merge.

2018-09-19 Thread hanm
GitHub user hanm opened a pull request: https://github.com/apache/zookeeper/pull/635 ZOOKEEPER-3151: Test build bot 2. Don't merge. You can merge this pull request into a Git repository by running: $ git pull https://github.com/hanm/zookeeper ZOOKEEPER-3151-2 Alternatively

[jira] [Updated] (ZOOKEEPER-3151) Jenkins github integration is broken if retriggering the precommit job through Jenkins admin web page.

2018-09-19 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3151?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-3151: -- Labels: pull-request-available (was: ) > Jenkins github integration is broken if

[GitHub] zookeeper pull request #634: ZOOKEEPER-3151: Test build bot 1. Don't merge.

2018-09-19 Thread hanm
GitHub user hanm opened a pull request: https://github.com/apache/zookeeper/pull/634 ZOOKEEPER-3151: Test build bot 1. Don't merge. You can merge this pull request into a Git repository by running: $ git pull https://github.com/hanm/zookeeper ZOOKEEPER-3151-1 Alternatively

Failed: ZOOKEEPER- PreCommit Build #2200

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

[jira] [Updated] (ZOOKEEPER-3032) Step 1.6 - Create zk-server maven structure

2018-09-19 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3032?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-3032: -- Labels: pull-request-available (was: ) > Step 1.6 - Create zk-server maven

[GitHub] zookeeper pull request #633: ZOOKEEPER-3032 - MAVEN MIGRATION - move java se...

2018-09-19 Thread nkalmar
GitHub user nkalmar opened a pull request: https://github.com/apache/zookeeper/pull/633 ZOOKEEPER-3032 - MAVEN MIGRATION - move java server, client You can merge this pull request into a Git repository by running: $ git pull https://github.com/nkalmar/zookeeper ZOOKEEPER-3032

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

2018-09-19 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper-trunk-windows-cmake/2943/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 9.74 KB...] [ivy:retrieve] confs: [javacc]

[jira] [Created] (ZOOKEEPER-3151) Jenkins github integration is broken if retriggering the precommit job through Jenkins admin web page.

2018-09-19 Thread Michael Han (JIRA)
Michael Han created ZOOKEEPER-3151: -- Summary: Jenkins github integration is broken if retriggering the precommit job through Jenkins admin web page. Key: ZOOKEEPER-3151 URL:

[GitHub] zookeeper issue #623: [ZOOKEEPER-3146] Limit the maximum client connections ...

2018-09-19 Thread hanm
Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/623 The [job I triggered on Sep 14](https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2168/) actually succeeded. It's the github integration that's broken. I suspect it's some

[jira] [Commented] (ZOOKEEPER-1177) Enabling a large number of watches for a large number of clients

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-1177?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16620889#comment-16620889 ] Hadoop QA commented on ZOOKEEPER-1177: -- -1 overall. GitHub Pull Request Build +1

Failed: ZOOKEEPER- PreCommit Build #2199

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

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

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/590 Rebased to resolve conflict. @anmolnar @hanm @maoling @nkalmar please revisit this PR when you have time. ---

[GitHub] zookeeper issue #610: [ZOOKEEPER-3124] Add the correct comment to show why w...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/610 With #622, we may actually not need the setCVersionPzxid logic anymore, any opinions on this? ---

[GitHub] zookeeper pull request #622: [ZOOKEEPER-3145] Fix potential watch missing is...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/622#discussion_r218880876 --- Diff: src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java --- @@ -1861,6 +1862,76 @@ public void

[GitHub] zookeeper issue #623: [ZOOKEEPER-3146] Limit the maximum client connections ...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/623 @hanm looks like the Jenkins job is pending there since you triggered it 5 days ago, it doesn't likely will be run, can you trigger it again? ---

[GitHub] zookeeper issue #605: [ZOOKEEPER-3125] Fixing pzxid consistent issue when re...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/605 @anmolnar after #631 being merged into 3.5, this patch can be cherry-picked back to 3.5 without conflict, can you help cherry-pick it back? ---

[GitHub] zookeeper issue #631: [ZOOKEEPER-3127] Fixing potential data inconsistency d...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/631 Thanks @anmolnar, close after merged. ---

[GitHub] zookeeper pull request #631: [ZOOKEEPER-3127] Fixing potential data inconsis...

2018-09-19 Thread lvfangmin
Github user lvfangmin closed the pull request at: https://github.com/apache/zookeeper/pull/631 ---

[jira] [Commented] (ZOOKEEPER-2641) AvgRequestLatency metric improves to be more accurate

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2641?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16620709#comment-16620709 ] Hadoop QA commented on ZOOKEEPER-2641: -- -1 overall. GitHub Pull Request Build +1

Failed: ZOOKEEPER- PreCommit Build #2198

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

[jira] [Commented] (ZOOKEEPER-3139) Zookeeper is not getting started because server folder is not present in the jre/bin for version 1.8.0_181 on 32 bit machine

2018-09-19 Thread maoling (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16620512#comment-16620512 ] maoling commented on ZOOKEEPER-3139: [~nitish.kulkarni] it may be a issue about your environment?

ZooKeeper_branch34_openjdk8 - Build # 60 - Failure

2018-09-19 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch34_openjdk8/60/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 16.83 KB...] at

ZooKeeper_branch35_jdk8 - Build # 1123 - Still Failing

2018-09-19 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch35_jdk8/1123/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 63.08 KB...] [junit] Running

[jira] [Commented] (ZOOKEEPER-3127) Fixing potential data inconsistency due to update last processed zxid with partial multi-op txn

2018-09-19 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16620402#comment-16620402 ] Andor Molnar commented on ZOOKEEPER-3127: - [~lvfangmin] I'm sorry, I didn't notice your

[jira] [Updated] (ZOOKEEPER-3127) Fixing potential data inconsistency due to update last processed zxid with partial multi-op txn

2018-09-19 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3127?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andor Molnar updated ZOOKEEPER-3127: Fix Version/s: 3.5.5 > Fixing potential data inconsistency due to update last

[GitHub] zookeeper issue #631: [ZOOKEEPER-3127] Fixing potential data inconsistency d...

2018-09-19 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/631 Merged to branch-3.5 Thanks @lvfangmin ! Please close this PR. ---

[GitHub] zookeeper pull request #630: ZOOKEEPER-2284:LogFormatter and SnapshotFormatt...

2018-09-19 Thread eolivelli
Github user eolivelli commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/630#discussion_r218679868 --- Diff: src/java/test/org/apache/zookeeper/ZKUtilTest.java --- @@ -0,0 +1,88 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

Success: ZOOKEEPER- PreCommit Build #2197

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

[jira] [Commented] (ZOOKEEPER-3150) Data integrity check when loading snapshot/txns from disk

2018-09-19 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3150?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16620152#comment-16620152 ] Hadoop QA commented on ZOOKEEPER-3150: -- +1 overall. GitHub Pull Request Build +1

[GitHub] zookeeper pull request #629: ZOOKEEPER-2641:AvgRequestLatency metric improve...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/629#discussion_r218674973 --- Diff: src/java/test/org/apache/zookeeper/server/ServerStatsTest.java --- @@ -61,7 +62,7 @@ public void testPacketsMetrics() {

[GitHub] zookeeper pull request #629: ZOOKEEPER-2641:AvgRequestLatency metric improve...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/629#discussion_r218676843 --- Diff: src/java/main/org/apache/zookeeper/server/NettyServerCnxn.java --- @@ -599,7 +599,7 @@ public void commandRun() { }

[GitHub] zookeeper pull request #629: ZOOKEEPER-2641:AvgRequestLatency metric improve...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/629#discussion_r218676806 --- Diff: src/java/main/org/apache/zookeeper/server/NIOServerCnxn.java --- @@ -819,7 +819,7 @@ public void commandRun() { }

[GitHub] zookeeper pull request #629: ZOOKEEPER-2641:AvgRequestLatency metric improve...

2018-09-19 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/629#discussion_r218676907 --- Diff: src/java/main/org/apache/zookeeper/server/ServerStats.java --- @@ -35,7 +35,7 @@ private long totalLatency = 0; private long