[jira] [Commented] (ZOOKEEPER-2807) Flaky test: org.apache.zookeeper.test.WatchEventWhenAutoResetTest.testNodeDataChanged

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278053#comment-16278053 ] ASF GitHub Bot commented on ZOOKEEPER-2807: --- Github user anmolnar commented on a diff in the

[GitHub] zookeeper pull request #300: ZOOKEEPER-2807: Flaky test: org.apache.zookeepe...

2017-12-04 Thread anmolnar
Github user anmolnar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/300#discussion_r154851878 --- Diff: src/java/main/org/apache/zookeeper/server/quorum/CommitProcessor.java --- @@ -240,84 +240,14 @@ public void run() { }

[jira] [Commented] (ZOOKEEPER-2807) Flaky test: org.apache.zookeeper.test.WatchEventWhenAutoResetTest.testNodeDataChanged

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16278048#comment-16278048 ] ASF GitHub Bot commented on ZOOKEEPER-2807: --- Github user anmolnar commented on the issue:

[GitHub] zookeeper issue #300: ZOOKEEPER-2807: Flaky test: org.apache.zookeeper.test....

2017-12-04 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/300 @afine You're right. I missed that syncWithLeader() call is on the same path and in the same thread as adding commits to the queue. In which case this must be right: -

[jira] [Commented] (ZOOKEEPER-2807) Flaky test: org.apache.zookeeper.test.WatchEventWhenAutoResetTest.testNodeDataChanged

2017-12-04 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277625#comment-16277625 ] Hadoop QA commented on ZOOKEEPER-2807: -- -1 overall. GitHub Pull Request Build +1

Failed: ZOOKEEPER- PreCommit Build #1333

2017-12-04 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/1333/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 87.12 MB...] [exec] +1

[jira] [Commented] (ZOOKEEPER-2807) Flaky test: org.apache.zookeeper.test.WatchEventWhenAutoResetTest.testNodeDataChanged

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277582#comment-16277582 ] ASF GitHub Bot commented on ZOOKEEPER-2807: --- Github user afine commented on a diff in the

[GitHub] zookeeper pull request #300: ZOOKEEPER-2807: Flaky test: org.apache.zookeepe...

2017-12-04 Thread afine
Github user afine commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/300#discussion_r154786233 --- Diff: src/java/main/org/apache/zookeeper/server/quorum/CommitProcessor.java --- @@ -327,6 +257,95 @@ public void run() {

Re: Proposal for reworked ZK web site generation: CMS -> jekyll

2017-12-04 Thread Patrick Hunt
Hi folks, final update on this one. INFRA has completed the switch over and we are now live on the new website generation process. I've updated the "how to release" page as well as the cwiki describing how our website is managed. https://cwiki.apache.org/confluence/display/ZOOKEEPER/WebSiteSetup

[jira] [Commented] (ZOOKEEPER-2807) Flaky test: org.apache.zookeeper.test.WatchEventWhenAutoResetTest.testNodeDataChanged

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277541#comment-16277541 ] ASF GitHub Bot commented on ZOOKEEPER-2807: --- Github user afine commented on the issue:

[GitHub] zookeeper issue #300: ZOOKEEPER-2807: Flaky test: org.apache.zookeeper.test....

2017-12-04 Thread afine
Github user afine commented on the issue: https://github.com/apache/zookeeper/pull/300 @anmolnar With respect to the code path above, shouldn't that be blocked on `syncWithLeader`? > Even if you drain the committedRequests, I'm not sure that guarantees that there are no more

Re: ZK Jenkins job updates.

2017-12-04 Thread Abraham Fine
Thank you Patrick! This should make debugging flaky tests much less painful. Abe On Sun, Dec 3, 2017, at 17:33, Patrick Hunt wrote: > I made a few updates to the jenkins job configs. Abe pointed out that > while > we are now generating individual logs files for tests (one per junit test >

[jira] [Commented] (ZOOKEEPER-2915) Use "strict" conflict management in ivy

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2915?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277327#comment-16277327 ] ASF GitHub Bot commented on ZOOKEEPER-2915: --- Github user afine closed the pull request at:

[GitHub] zookeeper pull request #426: ZOOKEEPER-2915: Use "strict" conflict managemen...

2017-12-04 Thread afine
Github user afine closed the pull request at: https://github.com/apache/zookeeper/pull/426 ---

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

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16276730#comment-16276730 ] ASF GitHub Bot commented on ZOOKEEPER-2184: --- Github user riccardofreixo commented on the

[GitHub] zookeeper issue #150: ZOOKEEPER-2184: Zookeeper Client should re-resolve hos...

2017-12-04 Thread riccardofreixo
Github user riccardofreixo commented on the issue: https://github.com/apache/zookeeper/pull/150 @sslavic thanks for the suggestion. We haven't tried that approach, and as far as I can tell it sounds like it would work. You'd still have the re-resolution problem if you

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

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16276711#comment-16276711 ] ASF GitHub Bot commented on ZOOKEEPER-2184: --- Github user sslavic commented on the issue:

[GitHub] zookeeper issue #150: ZOOKEEPER-2184: Zookeeper Client should re-resolve hos...

2017-12-04 Thread sslavic
Github user sslavic commented on the issue: https://github.com/apache/zookeeper/pull/150 @riccardofreixo have you tried using ClusterIP Service for ZooKeeper StatefulSet and providing that ClusterIP (or service hostname) to Kafka / ZooKeeper clients as sole ZooKeeper hostname?

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

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16276675#comment-16276675 ] ASF GitHub Bot commented on ZOOKEEPER-2184: --- Github user riccardofreixo commented on the

[GitHub] zookeeper issue #150: ZOOKEEPER-2184: Zookeeper Client should re-resolve hos...

2017-12-04 Thread riccardofreixo
Github user riccardofreixo commented on the issue: https://github.com/apache/zookeeper/pull/150 We're running Kafka in Kubernetes, so this bug was biting us regularly. We applied the patch in the kafka clusters of our client and are running in prod. Solves our problem and created

[GitHub] zookeeper issue #150: ZOOKEEPER-2184: Zookeeper Client should re-resolve hos...

2017-12-04 Thread jorgheymans
Github user jorgheymans commented on the issue: https://github.com/apache/zookeeper/pull/150 just got stung by this as well, assumed zk clients would be clever enough to reresolve :-/ Since there is a lot of interest in this why not just rebase-merge and let ppl test out

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

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16276584#comment-16276584 ] ASF GitHub Bot commented on ZOOKEEPER-2184: --- Github user jorgheymans commented on the issue:

[GitHub] zookeeper issue #415: ZOOKEEPER-2939: Added last/min/max proposal size JMX b...

2017-12-04 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/415 @afine @phunt Findbugs issues have been resolved. Please review & commit. ---

[jira] [Commented] (ZOOKEEPER-2939) Deal with maxbuffer as it relates to proposals

2017-12-04 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16276570#comment-16276570 ] ASF GitHub Bot commented on ZOOKEEPER-2939: --- Github user anmolnar commented on the issue: