[GitHub] zookeeper pull request #555: ZOOKEEPER-3061: add more details to 'Unhandled ...

2018-07-13 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/555#discussion_r202508110 --- Diff: src/java/main/org/apache/zookeeper/server/quorum/LearnerHandler.java --- @@ -792,7 +792,14 @@ public boolean syncFollower(long peerLastZxid,

[GitHub] zookeeper pull request #566: ZOOKEEPER-3062: mention fsync.warningthresholdm...

2018-07-13 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/566#discussion_r202507971 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java --- @@ -99,6 +99,9 @@ public static final String LOG_FILE

[GitHub] zookeeper pull request #566: ZOOKEEPER-3062: mention fsync.warningthresholdm...

2018-07-13 Thread lvfangmin
Github user lvfangmin commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/566#discussion_r202508017 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java --- @@ -339,7 +342,8 @@ public synchronized void commit() throws IOExcep

[GitHub] zookeeper issue #565: ZOOKEEPER-3067: Optionally disable client environment ...

2018-07-13 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/zookeeper/pull/565 Unrelated test failure https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/1936/testReport/org.apache.zookeeper.server.quorum/QuorumPeerMainTest/testFailedTxnAsPartOfQuorumLoss/ ---

[jira] [Commented] (ZOOKEEPER-3067) Optionally suppress client environment logging.

2018-07-13 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16543952#comment-16543952 ] Hadoop QA commented on ZOOKEEPER-3067: -- -1 overall. GitHub Pull Request Build

Failed: ZOOKEEPER- PreCommit Build #1936

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

[jira] [Commented] (ZOOKEEPER-3067) Optionally suppress client environment logging.

2018-07-13 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3067?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16543916#comment-16543916 ] Hadoop QA commented on ZOOKEEPER-3067: -- -1 overall. GitHub Pull Request Build

Failed: ZOOKEEPER- PreCommit Build #1935

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

[GitHub] zookeeper issue #565: ZOOKEEPER-3067: Optionally disable client environment ...

2018-07-13 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/565 @jpeach Issue was on master, but it's already fixed. Please trigger another build by amending your latest commit. ---

[GitHub] zookeeper issue #565: ZOOKEEPER-3067: Optionally disable client environment ...

2018-07-13 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/zookeeper/pull/565 AFAICT the test failure is from findbugs, which shouldn't have been affected in this commit. I couldn't find what it was happy about in the test output. ---

ZooKeeper_branch34_openjdk7 - Build # 1989 - Failure

2018-07-13 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch34_openjdk7/1989/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 39.82 KB...] [junit] Tests run: 1, Failures

[GitHub] zookeeper pull request #567: ZOOKEEPER-3071: Add a config parameter to contr...

2018-07-13 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/567#discussion_r202364176 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java --- @@ -347,6 +397,13 @@ public synchronized void commit() throws IOExcep

[GitHub] zookeeper pull request #567: ZOOKEEPER-3071: Add a config parameter to contr...

2018-07-13 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/567#discussion_r202363134 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java --- @@ -102,6 +102,20 @@ /** Maximum time we allow for elapsed f

[GitHub] zookeeper pull request #567: ZOOKEEPER-3071: Add a config parameter to contr...

2018-07-13 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/567#discussion_r202364713 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java --- @@ -347,6 +397,13 @@ public synchronized void commit() throws IOExcep

[GitHub] zookeeper pull request #567: ZOOKEEPER-3071: Add a config parameter to contr...

2018-07-13 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/567#discussion_r202363520 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java --- @@ -110,6 +124,18 @@ if ((fsyncWarningThreshold = Long.

[GitHub] zookeeper pull request #567: ZOOKEEPER-3071: Add a config parameter to contr...

2018-07-13 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/567#discussion_r202362761 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java --- @@ -110,6 +124,18 @@ if ((fsyncWarningThreshold = Long.

[GitHub] zookeeper pull request #567: ZOOKEEPER-3071: Add a config parameter to contr...

2018-07-13 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/567#discussion_r202364057 --- Diff: src/java/main/org/apache/zookeeper/server/persistence/FileTxnLog.java --- @@ -347,6 +397,13 @@ public synchronized void commit() throws IOExcep

ZooKeeper_branch35_jdk8 - Build # 1038 - Still Failing

2018-07-13 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch35_jdk8/1038/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 62.43 KB...] [junit] Running org.apache.zookeep

ZooKeeper-trunk - Build # 100 - Still Failing

2018-07-13 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper-trunk/100/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 3.94 KB...] at com.google.common.util.concurrent.Fo

[jira] [Resolved] (ZOOKEEPER-3033) Step 1.2 - Create zk-recipes maven structure

2018-07-13 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3033?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andor Molnar resolved ZOOKEEPER-3033. - Resolution: Fixed Fix Version/s: 3.5.5 3.6.0 Issue resolved

[GitHub] zookeeper issue #564: ZOOKEEPER-3033 - MAVEN MIGRATION - Step 1.2 - create z...

2018-07-13 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/564 Merged. Thanks @nkalmar ---

[jira] [Updated] (ZOOKEEPER-3033) Step 1.2 - Create zk-recipes maven structure

2018-07-13 Thread Norbert Kalmar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3033?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norbert Kalmar updated ZOOKEEPER-3033: -- Affects Version/s: 3.5.4 3.4.12 > Step 1.2 - Create zk-recip

[GitHub] zookeeper pull request #564: ZOOKEEPER-3033 - MAVEN MIGRATION - Step 1.2 - c...

2018-07-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zookeeper/pull/564 ---

ZooKeeper-trunk - Build # 99 - Failure

2018-07-13 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper-trunk/99/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 142.58 KB...] [junit] Tests run: 1, Failures: 0, Errors:

Success: ZOOKEEPER- PreCommit Build #1934

2018-07-13 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/1934/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 85.97 MB...] [exec] +1 @aut

[jira] [Commented] (ZOOKEEPER-2184) Zookeeper Client should re-resolve hosts when connection attempts fail

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

[GitHub] zookeeper issue #556: ZOOKEEPER-3074:Flaky test:org.apache.zookeeper.server....

2018-07-13 Thread maoling
Github user maoling commented on the issue: https://github.com/apache/zookeeper/pull/556 It seems that Jenkins doesn't like me, even doesn't reject me,let me wait and wait ,just like some girls I chase. ---

[GitHub] zookeeper issue #553: continue can be replaced with break

2018-07-13 Thread maoling
Github user maoling commented on the issue: https://github.com/apache/zookeeper/pull/553 @a470577391 it's useful and enough to change `continue` to `break`,especially when call `getLogFiles(logDir.listFiles(), 0)` It's better to create a [JIRA](https://issues.apache.org/jira/

[jira] [Resolved] (ZOOKEEPER-3087) Fix findbug warning introduced by ZOOKEEPER-3084.

2018-07-13 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3087?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andor Molnar resolved ZOOKEEPER-3087. - Resolution: Fixed Fix Version/s: 3.6.0 Issue resolved by pull request 568 [htt

[GitHub] zookeeper pull request #568: ZOOKEEPER-3087: Fix findbug warning introduced ...

2018-07-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zookeeper/pull/568 ---