Re: Zookeeper at Twitter by Micheal

2018-10-18 Thread Rakesh Radhakrishnan
Great article Michael. Appreciate all the efforts in making this blog! Cheers, Rakesh On Mon, Oct 15, 2018 at 1:56 PM Norbert Kalmar wrote: > Good blog post! > Can't wait for the PRs ;) > I'm very positive about ZooKeeper's future (heard lot of talks lately about > etcd overthrowing ZooKeeper

[GitHub] zookeeper issue #673: [ZOOKEEPER-3177] Refactor request throttle logic in NI...

2018-10-18 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/673 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2475/ ---

Failed: ZOOKEEPER- PreCommit Build #2475

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

Re: Zookeeper at Twitter by Micheal

2018-10-18 Thread Michael Han
Thank you all for the feedback, glad to see the blog post is well received. On Mon, Oct 15, 2018 at 1:26 AM Norbert Kalmar wrote: > Good blog post! > Can't wait for the PRs ;) > I'm very positive about ZooKeeper's future (heard lot of talks lately about > etcd overthrowing ZooKeeper - no chance

Re: Decrease number of threads in Jenkins builds to reduce flakyness

2018-10-18 Thread Michael Han
It's a good idea to reduce the concurrency of to eliminate flakyness. Looks like single threaded unit tests on trunk is pretty stable https://builds.apache.org/job/zookeeper-trunk-single-thread/ (some failures are due to C tests). The build time is longer, but not too bad (for pre-commit build,

Re: Decrease number of threads in Jenkins builds to reduce flakyness

2018-10-18 Thread Andor Molnar
Lots of ConnectionLoss and Address already in use failures on branch34_java9. Looks like specific to Jenkins slave H22. Andor On Mon, Oct 15, 2018 at 2:50 PM, Andor Molnar wrote: > +1 > > > > On Mon, Oct 15, 2018 at 1:55 PM, Enrico Olivelli > wrote: > >> Il giorno lun 15 ott 2018 alle ore

[GitHub] zookeeper issue #651: ZOOKEEPER-3113 EphemeralType.get() fails to verify eph...

2018-10-18 Thread hanm
Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/651 one comment - it would be good for the updated test of EphemeralType.get to cover the code path of throwing an IllegalArgumentException. This was what I referred to in my original comment about

[jira] [Updated] (ZOOKEEPER-3177) Refactor request throttle logic in NIO and Netty to keep the same behavior and make the code easier to maintain

2018-10-18 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3177?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-3177: -- Labels: pull-request-available (was: ) > Refactor request throttle logic in NIO

[GitHub] zookeeper pull request #673: [ZOOKEEPER-3177] Refactor request throttle logi...

2018-10-18 Thread lvfangmin
GitHub user lvfangmin opened a pull request: https://github.com/apache/zookeeper/pull/673 [ZOOKEEPER-3177] Refactor request throttle logic in NIO and Netty to keep the same behavior and make the code easier to maintain You can merge this pull request into a Git repository by

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

2018-10-18 Thread hanm
Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/647 I think we can add Retry rules as long as the cause of the flaky is clear (e.g. this case, since ConnectionLoss is a well known flaky cause); what I worried previously was to apply it unanimously

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

2018-10-18 Thread asfgit
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/2474/ ---

Failed: ZOOKEEPER- PreCommit Build #2474

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

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

2018-10-18 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/647 @anmolnar I can understand your concern, let's remove the RetryRule for now, we can add it when necessary. ---

[GitHub] zookeeper issue #669: ZOOKEEPER-3152: Port ZK netty stack to netty4

2018-10-18 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/669 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2473/ ---

[jira] [Commented] (ZOOKEEPER-3113) EphemeralType.get() fails to verify ephemeralOwner when currentElapsedTime() is small enough

2018-10-18 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16655504#comment-16655504 ] Hudson commented on ZOOKEEPER-3113: --- FAILURE: Integrated in Jenkins build ZooKeeper-trunk #238 (See

ZooKeeper-trunk - Build # 238 - Still Failing

2018-10-18 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper-trunk/238/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 334.36 KB...] [exec] Log Message Received: [2018-10-18

Re: Wiki access

2018-10-18 Thread Patrick Hunt
Send me your wiki ID privately and I will set it up for you. Patrick On Thu, Oct 18, 2018 at 7:35 AM Tamas Penzes wrote: > Hi All, > > Could you please give me permission to edit the wiki. > I should update a few thing related to doc generation. > > Thanks, Tamaas >

ZooKeeper_branch35_jdk8 - Build # 1161 - Failure

2018-10-18 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch35_jdk8/1161/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 63.95 KB...] [junit] Tests run: 3, Failures:

Wiki access

2018-10-18 Thread Tamas Penzes
Hi All, Could you please give me permission to edit the wiki. I should update a few thing related to doc generation. Thanks, Tamaas

Success: ZOOKEEPER- PreCommit Build #2472

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

[GitHub] zookeeper issue #671: ZOOKEEPER-3155: Remove Forrest XMLs and their build pr...

2018-10-18 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/671 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2472/ ---

[GitHub] zookeeper issue #662: ZOOKEEPER-3162. Broken lock semantics in C client lock...

2018-10-18 Thread tamaashu
Github user tamaashu commented on the issue: https://github.com/apache/zookeeper/pull/662 looks okay, thanks for fixing build issues too ---

[GitHub] zookeeper issue #665: [ZOOKEEPER-3163] Use session map in the Netty to impro...

2018-10-18 Thread tamaashu
Github user tamaashu commented on the issue: https://github.com/apache/zookeeper/pull/665 the change looks clean and logical, ready to commit ---

[GitHub] zookeeper issue #671: ZOOKEEPER-3155: Remove Forrest XMLs and their build pr...

2018-10-18 Thread tamaashu
Github user tamaashu commented on the issue: https://github.com/apache/zookeeper/pull/671 retest this please ---

[GitHub] zookeeper issue #672: ZOOKEEPER-3032 - MAVEN MIGRATION - zookeeper-server

2018-10-18 Thread tamaashu
Github user tamaashu commented on the issue: https://github.com/apache/zookeeper/pull/672 Looks okay. The separation of client and server can wait for ZooKeeper 4.0, where we can break compatibility. ---

[GitHub] zookeeper issue #654: ZOOKEEPER-3160: Custom User SSLContext

2018-10-18 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/654 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2471/ ---

[GitHub] zookeeper issue #654: ZOOKEEPER-3160: Custom User SSLContext

2018-10-18 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/zookeeper/pull/654 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2470/ ---

Failed: ZOOKEEPER- PreCommit Build #2470

2018-10-18 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2470/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 75.67 MB...] [exec] /bin/kill

Failed: ZOOKEEPER- PreCommit Build #2471

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

[GitHub] zookeeper issue #654: ZOOKEEPER-3160: Custom User SSLContext

2018-10-18 Thread arankin-irl
Github user arankin-irl commented on the issue: https://github.com/apache/zookeeper/pull/654 Merged the latest 3.5 branch in - looks like there was a significant structure rework, along with some changes to the ZKConfig properties - moved the additional property in with the others in

[jira] [Commented] (ZOOKEEPER-3113) EphemeralType.get() fails to verify ephemeralOwner when currentElapsedTime() is small enough

2018-10-18 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16655059#comment-16655059 ] Hudson commented on ZOOKEEPER-3113: --- SUCCESS: Integrated in Jenkins build

[GitHub] zookeeper pull request #670: DO_NOT_MERGE - MAVEN MIGRATION - Separation of ...

2018-10-18 Thread nkalmar
Github user nkalmar closed the pull request at: https://github.com/apache/zookeeper/pull/670 ---

[jira] [Resolved] (ZOOKEEPER-3113) EphemeralType.get() fails to verify ephemeralOwner when currentElapsedTime() is small enough

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

[GitHub] zookeeper issue #651: ZOOKEEPER-3113 EphemeralType.get() fails to verify eph...

2018-10-18 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/651 Committed. Thanks for the reviews. @hanm If you still do have any concerns, please let me know, I'll address them in a separate patch. ---

[GitHub] zookeeper pull request #651: ZOOKEEPER-3113 EphemeralType.get() fails to ver...

2018-10-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zookeeper/pull/651 ---