Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/647
@lvfangmin Given that I've already provided a fix for the flakyness in #657
, do we still need this retry rule?
---
Hi,
Let me summarize and give a quick update on the outstanding issues for 3.5 GA:
- ZOOKEEPER-1818 (Fix don't care for trunk)
- ZOOKEEPER-2778 (Potential server deadlock between follower sync with leader
and follower receiving external connection requests.)
- ZOOKEEPER-3021 Migrate project
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2979?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated ZOOKEEPER-2979:
--
Labels: pull-request-available (was: )
> Use dropwizard library histogram for
Github user anmolnar closed the pull request at:
https://github.com/apache/zookeeper/pull/440
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/656
This is merged. Thanks @nkalmar !
Please close this PR.
---
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3161?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated ZOOKEEPER-3161:
--
Labels: pull-request-available (was: )
> Refactor QuorumPeerMainTest.java: move
GitHub user anmolnar opened a pull request:
https://github.com/apache/zookeeper/pull/659
ZOOKEEPER-3161. Refactor QuorumPeerMainTest.java: move commonly used
functions to base class
Move the following methods to QuorumPeerTestBase.java:
tearDown()
LaunchServers()
See https://builds.apache.org/job/ZooKeeper_branch35_jdk8/1146/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 68.99 KB...]
[junit] Tests run: 1, Failures:
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3157?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andor Molnar resolved ZOOKEEPER-3157.
-
Resolution: Fixed
Issue resolved by pull request 657
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2383/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 77.96 MB...]
[exec]
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/587
Committed to master. Thanks @maoling !
This patch doesn't apply to branch-3.5, but I think we should fix the issue
on it too.
Would you please open a separate pull request?
---
Github user eolivelli commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/587#discussion_r223215411
--- Diff:
zookeeper-client/zookeeper-client-java/src/main/java/org/apache/zookeeper/client/ConnectStringParser.java
---
@@ -68,14 +71,16 @@ public
Github user asfgit closed the pull request at:
https://github.com/apache/zookeeper/pull/587
---
Github user asfgit closed the pull request at:
https://github.com/apache/zookeeper/pull/657
---
Andor Molnar created ZOOKEEPER-3161:
---
Summary: Refactor QuorumPeerMainTest.java: move commonly used
functions to base class
Key: ZOOKEEPER-3161
URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3161
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2380/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 79.66 MB...]
[exec]
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/655
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2380/
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/655
retest this please
---
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/651
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2382/
---
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2382/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 76.61 MB...]
[exec]
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/659
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2383/
---
Github user eolivelli commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/587#discussion_r223347154
--- Diff:
zookeeper-client/zookeeper-client-java/src/main/java/org/apache/zookeeper/client/ConnectStringParser.java
---
@@ -68,14 +71,16 @@ public
Github user eolivelli commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/587#discussion_r223346987
--- Diff:
zookeeper-client/zookeeper-client-java/src/main/java/org/apache/zookeeper/client/ConnectStringParser.java
---
@@ -68,14 +71,16 @@ public
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/657
Apologies, committed my own patch. Test fix, got 2 approvals, one from the
original author.
---
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2381/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 82.71 MB...]
[exec]
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/653
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2381/
---
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2384/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 80.55 MB...]
[exec]
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/655
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2384/
---
See https://builds.apache.org/job/ZooKeeper_branch35_jdk8/1147/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 63.98 KB...]
[junit] Running
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3157?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16642002#comment-16642002
]
Hudson commented on ZOOKEEPER-3157:
---
FAILURE: Integrated in Jenkins build ZooKeeper-trunk #223 (See
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3106?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16642001#comment-16642001
]
Hudson commented on ZOOKEEPER-3106:
---
FAILURE: Integrated in Jenkins build ZooKeeper-trunk #223 (See
See https://builds.apache.org/job/ZooKeeper-trunk/223/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 196.43 KB...]
[junit] Tests run: 5, Failures: 0,
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/632
retest this please
---
Github user lvfangmin commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/632#discussion_r223538616
--- Diff:
zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java ---
@@ -1521,4 +1562,179 @@ public boolean removeWatch(String path,
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/647
I remember I commented in Jira ZOOKEEPER-3157, not sure why it didn't show
up.
I mentioned that we still need RetryRule, because there might be temporary
quorum unstable issues like
See https://builds.apache.org/job/ZooKeeper-trunk/224/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 332.71 KB...]
[exec] : elapsed 1001 : OK
[exec]
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/660
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2386/
---
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2386/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 76.25 MB...]
[exec]
Github user lvfangmin commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/659#discussion_r223564880
--- Diff:
zookeeper-server/src/test/java/org/apache/zookeeper/server/quorum/QuorumPeerTestBase.java
---
@@ -402,4 +421,129 @@ public File
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/659
retest this please
---
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/2391/
---
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2388/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 76.42 MB...]
[exec]
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/632
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2388/
---
Github user edwardoliveira commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/632#discussion_r223546118
--- Diff:
zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java ---
@@ -1521,4 +1562,179 @@ public boolean removeWatch(String
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2320?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16642802#comment-16642802
]
Hadoop QA commented on ZOOKEEPER-2320:
--
-1 overall. Here are the results of testing the latest
Jira: https://issues.apache.org/jira/browse/ZOOKEEPER-2320
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/3705/
###
## LAST 60 LINES OF THE CONSOLE
###
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/660
retest this please
---
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/660
@anmolnar do you know why it failing the check here? From the latest
Jenkins build link I cannot find any failure.
---
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/658
retest this please
---
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2391/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 149.52 KB...]
[exec]
Github user maoling commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/587#discussion_r223567109
--- Diff:
zookeeper-client/zookeeper-client-java/src/main/java/org/apache/zookeeper/client/ConnectStringParser.java
---
@@ -68,14 +71,16 @@ public
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2385/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 81.21 MB...]
[exec] +1
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2389/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 7.42 KB...]
[exec] Testing
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/647
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2389/
---
GitHub user anmolnar opened a pull request:
https://github.com/apache/zookeeper/pull/660
ZOOKEEPER-2320. C-client crashes when removing watcher asynchronously in
"local" mode
Trying to fix invalid point issue happening intermittently in the PreCommit
job. Looks like this old patch
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2320?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated ZOOKEEPER-2320:
--
Labels: pull-request-available (was: )
> C-client crashes when removing watcher
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/632
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2385/
---
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2387/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 69.09 MB...]
[exec]
Github user asfgit commented on the issue:
https://github.com/apache/zookeeper/pull/660
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2387/
---
59 matches
Mail list logo
