Github user lvfangmin commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/590#discussion_r216912996
--- Diff: src/java/test/org/apache/zookeeper/server/DumbWatcher.java ---
@@ -0,0 +1,96 @@
+/**
+ * Licensed to the Apache Software Foundation (AS
Github user lvfangmin commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/590#discussion_r216910219
--- Diff:
src/java/main/org/apache/zookeeper/server/watch/WatchManagerOptimized.java ---
@@ -0,0 +1,355 @@
+/**
+ * Licensed to the Apache Softw
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2990?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16611596#comment-16611596
]
Bogdan Kanivets commented on ZOOKEEPER-2990:
[~andorm] Sounds good. I'll
See https://builds.apache.org/job/ZooKeeper_branch35_jdk8/1115/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 100.96 KB...]
[junit] Running org.apache.zookee
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2153/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 80.35 MB...]
[exec]
[exec
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3144?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16611384#comment-16611384
]
Hadoop QA commented on ZOOKEEPER-3144:
--
-1 overall. GitHub Pull Request Build
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/605
Just reported another pzxid inconsistent issue related to CloseSession, I
found that issue recently, will wait this being merged before providing that
patch, which will make adding the unit test
Fangmin Lv created ZOOKEEPER-3145:
-
Summary: Potential watch missing issue due to stale pzxid when
replaying CloseSession txn with fuzzy snapshhot
Key: ZOOKEEPER-3145
URL: https://issues.apache.org/jira/browse/ZOO
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3144?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated ZOOKEEPER-3144:
--
Labels: pull-request-available (was: )
> Potential ephemeral nodes inconsistent du
GitHub user lvfangmin opened a pull request:
https://github.com/apache/zookeeper/pull/621
[ZOOKEEPER-3144] Fix potential ephemeral nodes inconsistent due to global
session inconsistent with fuzzy snapshot
There is a race condition between update the lastProcessedZxid and the
actual
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3144?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Fangmin Lv updated ZOOKEEPER-3144:
--
Summary: Potential ephemeral nodes inconsistent due to global session
inconsistent with fuz
Fangmin Lv created ZOOKEEPER-3144:
-
Summary: Potential ephemeral nodes inconsistent due to global
session inconsistent after with fuzzy snapshot
Key: ZOOKEEPER-3144
URL: https://issues.apache.org/jira/browse/ZOOKE
Hi Andor,
That's the on disk txn feature, which was disabled internally after we
found the potentially inconsistent issue. The only solution we have for now
is waiting for the new digest checking feature I mentioned in
ZOOKEEPER-3114.
I think there are some other critical consistent issues we jus
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2152/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 80.26 MB...]
[exec] +1 @aut
Github user hanm commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/619#discussion_r216842003
--- Diff: src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java
---
@@ -34,79 +36,133 @@
import org.apache.yetus.audience.InterfaceAudience;
Github user hanm commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/619#discussion_r216841877
--- Diff: src/java/main/org/apache/zookeeper/server/SnapshotFormatter.java
---
@@ -34,79 +36,133 @@
import org.apache.yetus.audience.InterfaceAudience;
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2261?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Han updated ZOOKEEPER-2261:
---
Fix Version/s: 3.5.5
> When only secureClientPort is configured connections, configuratio
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16611321#comment-16611321
]
Michael Han commented on ZOOKEEPER-2261:
I cherry picked the commit to 3.5 ju
Hi Jeelani,
Thanks for letting me know. I'm happy to remove it from the list to get
closer to a stable release. :)
What's the feature which can be disabled to avoid data inconsistency?
Andor
On 09/10/2018 11:33 PM, Mohamed Jeelani wrote:
> Thanks Andor for compiling this. Should we be ignor
See https://builds.apache.org/job/ZooKeeper-trunk-windows-cmake/2931/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 9.68 KB...]
[ivy:retrieve] confs: [javacc]
[
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2151/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 81.29 MB...]
[exec] +1 @aut
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3125?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16611041#comment-16611041
]
Hadoop QA commented on ZOOKEEPER-3125:
--
+1 overall. GitHub Pull Request Build
My primary concern is the drop of support and security patches for Java8 in
January by Oracle. It will require paying for support if not upgraded to Java
11. We are trying to get ahead of this because of budgets.
Jeremiah Adams
Software Engineer
www.helixeducation.com
Blog | Twitter | Facebook
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/605
Just found there is conflict to merge, I'm updating it now.
Btw, I just found 2 other potential data inconsistent issues about multi-op
and ephemeral, will create Jiras and send out diff
Github user lvfangmin commented on the issue:
https://github.com/apache/zookeeper/pull/605
This could cause watch event not notified when setWatches because of stale
pzxid, and all the tests have been passed, I think we should land this and port
it back to other branches as early as p
Hi Jeremiah. It's failing consistently on Jenkins, unlikely to officially
support until someone addresses those:
https://builds.apache.org/view/S-Z/view/ZooKeeper/job/ZooKeeper-trunk-java11/
We've been testing with openjdk for quite some time, those are supported.
The docs are ambiguous in that re
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16610791#comment-16610791
]
Hudson commented on ZOOKEEPER-3123:
---
SUCCESS: Integrated in Jenkins build ZooKeeper
We need your help to make the Apache Washington DC Roadshow on Dec 4th a
success.
What do we need most? Speakers!
We're bringing a unique DC flavor to this event by mixing Open Source
Software with talks about Apache projects as well as OSS CyberSecurity,
OSS in Government and and OSS Career
Hi,
I would like to share the plan for the new "Pluggable metrics system for
ZooKeeper" [1]
Currently on master we have the MetricsProvider API and the management for
the lifecycle of the MetricsProvider (start/stop/configuration).
Next immediate steps, which can run concurrently:
- Implemen
Enrico Olivelli created ZOOKEEPER-3143:
--
Summary: Pluggable metrics system for ZooKeeper -Data Collection
on Server
Key: ZOOKEEPER-3143
URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3143
P
See https://builds.apache.org/job/ZooKeeper-trunk-windows-cmake/2930/
###
## LAST 60 LINES OF THE CONSOLE
###
[...truncated 9.68 KB...]
[ivy:retrieve] confs: [javacc]
[
Github user asfgit closed the pull request at:
https://github.com/apache/zookeeper/pull/601
---
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/601
Merged. Thanks @eolivelli !
---
[
https://issues.apache.org/jira/browse/ZOOKEEPER-3123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Andor Molnar resolved ZOOKEEPER-3123.
-
Resolution: Fixed
Issue resolved by pull request 601
[https://github.com/apache/zooke
Github user anmolnar commented on the issue:
https://github.com/apache/zookeeper/pull/601
Latest build is green. I'll commit this one.
---
Github user anmolnar commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/601#discussion_r216663176
--- Diff:
src/java/test/org/apache/zookeeper/metrics/BaseTestMetricsProvider.java ---
@@ -0,0 +1,137 @@
+/**
+ * Licensed to the Apache Software
Github user eolivelli commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/601#discussion_r216658766
--- Diff:
src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java ---
@@ -1486,6 +1490,377 @@ public TestQPMain getTestQPMain() {
Github user eolivelli commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/601#discussion_r216657154
--- Diff:
src/java/test/org/apache/zookeeper/metrics/BaseTestMetricsProvider.java ---
@@ -0,0 +1,137 @@
+/**
+ * Licensed to the Apache Software
Github user anmolnar commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/601#discussion_r216653165
--- Diff:
src/java/test/org/apache/zookeeper/metrics/BaseTestMetricsProvider.java ---
@@ -0,0 +1,137 @@
+/**
+ * Licensed to the Apache Software
Github user anmolnar commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/601#discussion_r216654230
--- Diff:
src/java/test/org/apache/zookeeper/server/quorum/QuorumPeerMainTest.java ---
@@ -1486,6 +1490,377 @@ public TestQPMain getTestQPMain() {
Github user eolivelli commented on the issue:
https://github.com/apache/zookeeper/pull/601
@anmolnar @lvfangmin I think this is good to go
---
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16610514#comment-16610514
]
Andor Molnar edited comment on ZOOKEEPER-2261 at 9/11/18 12:21 PM:
[
https://issues.apache.org/jira/browse/ZOOKEEPER-2261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16610514#comment-16610514
]
Andor Molnar commented on ZOOKEEPER-2261:
-
Thanks [~hanm]
It's probably my f
43 matches
Mail list logo