[jira] [Commented] (ZOOKEEPER-2925) ZooKeeper server fails to start on first-startup due to race to create dataDir & snapDir

2018-10-16 Thread Mohammad Arshad (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16652982#comment-16652982 ] Mohammad Arshad commented on ZOOKEEPER-2925: [~dineshappavoo], no objecti

[jira] [Commented] (ZOOKEEPER-2925) ZooKeeper server fails to start on first-startup due to race to create dataDir & snapDir

2018-10-16 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16652966#comment-16652966 ] Andor Molnar commented on ZOOKEEPER-2925: - Hi [~dineshappavoo] Welcome to the

[jira] [Commented] (ZOOKEEPER-2925) ZooKeeper server fails to start on first-startup due to race to create dataDir & snapDir

2018-10-16 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16652830#comment-16652830 ] Hadoop QA commented on ZOOKEEPER-2925: -- -1 overall. Here are the results of tes

Failed: ZOOKEEPER-2925 PreCommit Build #3706

2018-10-16 Thread Apache Jenkins Server
Jira: https://issues.apache.org/jira/browse/ZOOKEEPER-2925 Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/3706/ ### ## LAST 60 LINES OF THE CONSOLE ### [

ZooKeeper_branch34_jdk8 - Build # 1565 - Failure

2018-10-16 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch34_jdk8/1565/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 42.55 KB...] [junit] Running org.apache.zookeep

[jira] [Comment Edited] (ZOOKEEPER-2925) ZooKeeper server fails to start on first-startup due to race to create dataDir & snapDir

2018-10-16 Thread Dinesh Appavoo (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16652716#comment-16652716 ] Dinesh Appavoo edited comment on ZOOKEEPER-2925 at 10/17/18 12:24 AM: -

[jira] [Commented] (ZOOKEEPER-2925) ZooKeeper server fails to start on first-startup due to race to create dataDir & snapDir

2018-10-16 Thread Dinesh Appavoo (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2925?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16652716#comment-16652716 ] Dinesh Appavoo commented on ZOOKEEPER-2925: --- Looks like this issue is not a

[GitHub] zookeeper issue #667: ZOOKEEPER-3158:firstConnect.countDown() will not be ex...

2018-10-16 Thread dineshappavoo
Github user dineshappavoo commented on the issue: https://github.com/apache/zookeeper/pull/667 This change LGTM 👍 ---

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

2018-10-16 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/2463/ ---

Failed: ZOOKEEPER- PreCommit Build #2463

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

[jira] [Commented] (ZOOKEEPER-236) SSL Support for Atomic Broadcast protocol

2018-10-16 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16652570#comment-16652570 ] Hudson commented on ZOOKEEPER-236: -- SUCCESS: Integrated in Jenkins build ZooKeeper-tr

[GitHub] zookeeper issue #567: ZOOKEEPER-3071: Add a config parameter to control tran...

2018-10-16 Thread suyogmapara
Github user suyogmapara commented on the issue: https://github.com/apache/zookeeper/pull/567 Hi @anmolnar please take a look at updated documentation. This time after regeneration, other parts of the documentation has changed. I have also rebased on the latest upstream/master ---

[GitHub] zookeeper issue #567: ZOOKEEPER-3071: Add a config parameter to control tran...

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

Failed: ZOOKEEPER- PreCommit Build #2462

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

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

2018-10-16 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/150 @jeffwidman You're right. @fpj is able to close it. ---

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

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

Failed: ZOOKEEPER- PreCommit Build #2461

2018-10-16 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2461/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 6.22 KB...] NODE_NAME=H0 ghprbPullLo

[GitHub] zookeeper issue #151: Applying ZOOKEEPER-2642 patch to a 3.5 base

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

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

2018-10-16 Thread jeffwidman
Github user jeffwidman commented on the issue: https://github.com/apache/zookeeper/pull/150 IIUC, this PR should be closed in favor of #451, which has already been merged. ---

Failed: ZOOKEEPER- PreCommit Build #2460

2018-10-16 Thread Apache Jenkins Server
Build: https://builds.apache.org/job/PreCommit-ZOOKEEPER-github-pr-build/2460/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 5.92 KB...] at hudson.model.

[jira] [Updated] (ZOOKEEPER-2642) ZooKeeper reconfig API backward compatibility fix

2018-10-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2642?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-2642: -- Labels: pull-request-available (was: ) > ZooKeeper reconfig API backward compatibi

[GitHub] zookeeper issue #151: Applying ZOOKEEPER-2642 patch to a 3.5 base

2018-10-16 Thread jeffwidman
Github user jeffwidman commented on the issue: https://github.com/apache/zookeeper/pull/151 nudge @Randgalt ---

[GitHub] zookeeper issue #652: ZOOKEEPER-3156: Add in option to canonicalize host nam...

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

Success: ZOOKEEPER- PreCommit Build #2459

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

[jira] [Commented] (ZOOKEEPER-236) SSL Support for Atomic Broadcast protocol

2018-10-16 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16652015#comment-16652015 ] Hudson commented on ZOOKEEPER-236: -- SUCCESS: Integrated in Jenkins build Zookeeper-tr

[GitHub] zookeeper issue #652: ZOOKEEPER-3156: Add in option to canonicalize host nam...

2018-10-16 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/zookeeper/pull/652 retest this please ---

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

2018-10-16 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/647 I still have bad feelings about introducing `RetryRule` in this patch. I haven't seen connectionLoss errors recently on the builds, neither on this patch after the fix and not sure it's a good th

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

2018-10-16 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/2458/ ---

Success: ZOOKEEPER- PreCommit Build #2458

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

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

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

Success: ZOOKEEPER- PreCommit Build #2456

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

[GitHub] zookeeper issue #652: ZOOKEEPER-3156: Add in option to canonicalize host nam...

2018-10-16 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/652 @revans2 We might need another committer to take a look. Also you probably need to trigger another build to get a green one. ---

[jira] [Created] (ZOOKEEPER-3178) Remove PrepRequestProcessor from RO ZooKeeperServer to avoid txns being created in RO mode

2018-10-16 Thread Fangmin Lv (JIRA)
Fangmin Lv created ZOOKEEPER-3178: - Summary: Remove PrepRequestProcessor from RO ZooKeeperServer to avoid txns being created in RO mode Key: ZOOKEEPER-3178 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3178

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

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

Failed: ZOOKEEPER- PreCommit Build #2457

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

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

2018-10-16 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/665 retest this please ---

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

2018-10-16 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/665 The failure seems not related to this code change, I'll trigger another round of test, this should be ready to get in. ---

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

2018-10-16 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/2454/ ---

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

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

Failed: ZOOKEEPER- PreCommit Build #2455

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

Failed: ZOOKEEPER- PreCommit Build #2454

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

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

2018-10-16 Thread lvfangmin
Github user lvfangmin commented on the issue: https://github.com/apache/zookeeper/pull/647 retest this please ---

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

2018-10-16 Thread Fangmin Lv (JIRA)
Fangmin Lv created ZOOKEEPER-3177: - Summary: Refactor request throttle logic in NIO and Netty to keep the same behavior and make the code easier to maintain Key: ZOOKEEPER-3177 URL: https://issues.apache.org/jira/

[GitHub] zookeeper issue #652: ZOOKEEPER-3156: Add in option to canonicalize host nam...

2018-10-16 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/zookeeper/pull/652 Ping any hope in getting this merged in? ---

[GitHub] zookeeper issue #627: ZOOKEEPER-236: SSL Support for Atomic Broadcast protoc...

2018-10-16 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/627 @ivmaykov It's in now. You can go ahead with the 2nd part. ---

[GitHub] zookeeper pull request #655: ZOOKEEPER-236: SSL Support for Atomic Broadcast...

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

Failed: ZOOKEEPER- PreCommit Build #2453

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

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

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

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

2018-10-16 Thread nkalmar
Github user nkalmar commented on the issue: https://github.com/apache/zookeeper/pull/672 retest this please ---

Failed: ZOOKEEPER- PreCommit Build #2452

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

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

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

ZooKeeper_branch34_openjdk8 - Build # 90 - Failure

2018-10-16 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch34_openjdk8/90/ ### ## LAST 60 LINES OF THE CONSOLE ### Started by timer [EnvInject] - Loading node environment variabl

Success: ZOOKEEPER- PreCommit Build #2449

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

[GitHub] zookeeper issue #655: ZOOKEEPER-236: SSL Support for Atomic Broadcast protoc...

2018-10-16 Thread asfgit
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/2449/ ---

Failed: ZOOKEEPER- PreCommit Build #2451

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

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

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

Failed: ZOOKEEPER- PreCommit Build #2450

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

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

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

[GitHub] zookeeper pull request #672: ZOOKEEPER-3032 - MAVEN MIGRATION - zookeeper-se...

2018-10-16 Thread nkalmar
GitHub user nkalmar opened a pull request: https://github.com/apache/zookeeper/pull/672 ZOOKEEPER-3032 - MAVEN MIGRATION - zookeeper-server Separating the java code is not feasible. Moving common and client back to server. You can merge this pull request into a Git repository by ru

[GitHub] zookeeper issue #655: ZOOKEEPER-236: SSL Support for Atomic Broadcast protoc...

2018-10-16 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/655 retest this please ---

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

2018-10-16 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/2448/ ---

Failed: ZOOKEEPER- PreCommit Build #2448

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

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

2018-10-16 Thread asfgit
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/2447/ ---

Success: ZOOKEEPER- PreCommit Build #2447

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

[GitHub] zookeeper issue #655: ZOOKEEPER-236: SSL Support for Atomic Broadcast protoc...

2018-10-16 Thread asfgit
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/2446/ ---

Failed: ZOOKEEPER- PreCommit Build #2446

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

[jira] [Updated] (ZOOKEEPER-3155) Remove Forrest XMLs and their build process from the project

2018-10-16 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3155?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-3155: -- Labels: pull-request-available (was: ) > Remove Forrest XMLs and their build proce

[GitHub] zookeeper pull request #671: ZOOKEEPER-3155: Remove Forrest XMLs and their b...

2018-10-16 Thread tamaashu
GitHub user tamaashu opened a pull request: https://github.com/apache/zookeeper/pull/671 ZOOKEEPER-3155: Remove Forrest XMLs and their build process from the … …project In this commit I've removed the forrest XML files and their result from the repository. I've also

Re: [VOTE] Maven migration - separation of java files to server and client project?

2018-10-16 Thread Andor Molnar
Thanks Norbert for taking care of this. No surprise here in a 10+ year old project. -1 (binding) for the separation Let’s keep server + client + common together for now. We can revisit this later, but the pro is that we keep the release artifact structure and not introducing breaking changes.

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

2018-10-16 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/651 retest this please ---

[jira] [Updated] (ZOOKEEPER-2711) Deadlock between concurrent 4LW commands that iterate over connections with Netty server

2018-10-16 Thread maoling (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-2711?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] maoling updated ZOOKEEPER-2711: --- Description: Observed the following issue in some $dayjob testing environments. Line numbers are

[GitHub] zookeeper issue #668: ZOOKEEPER-3165. Java 9: X509UtilTest.testCreateSSLCont...

2018-10-16 Thread anmolnar
Github user anmolnar commented on the issue: https://github.com/apache/zookeeper/pull/668 Committed to branch-3.5 ---

[jira] [Resolved] (ZOOKEEPER-3165) Java 9: X509UtilTest.testCreateSSLContextWithoutTrustStorePassword fails

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

[GitHub] zookeeper pull request #668: ZOOKEEPER-3165. Java 9: X509UtilTest.testCreate...

2018-10-16 Thread anmolnar
Github user anmolnar closed the pull request at: https://github.com/apache/zookeeper/pull/668 ---

[jira] [Reopened] (ZOOKEEPER-3032) Step 1.6 - Create zk-server maven structure

2018-10-16 Thread Norbert Kalmar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3032?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norbert Kalmar reopened ZOOKEEPER-3032: --- > Step 1.6 - Create zk-server maven structure > -

[jira] [Updated] (ZOOKEEPER-3032) Step 1.6 - Create zk-server maven structure

2018-10-16 Thread Norbert Kalmar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3032?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norbert Kalmar updated ZOOKEEPER-3032: -- Description: Create a project structure that separates the different parts of ZooKe

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

2018-10-16 Thread andreareale
Github user andreareale commented on the issue: https://github.com/apache/zookeeper/pull/662 Thanks @nkalmar. There were some additional path issues in the recipes ant test targets. Updated the PR to fix them as well. Please, let me know if there is anything I can do to help with merg

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

2018-10-16 Thread nkalmar
Github user nkalmar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/670#discussion_r225422359 --- Diff: zookeeper-client/zookeeper-client-java/src/main/java/org/apache/zookeeper/client/ZKClientConfig.java --- @@ -23,7 +23,7 @@ import org.apac

Re: [VOTE] Maven migration - separation of java files to server and client project?

2018-10-16 Thread Enrico Olivelli
Yes, I think it is NOT a good idea to go ahead with this separation. so -1 (non binding) from my side for now And your patch is very good at demonstrating this. We can't break compatibility in clients. We can move to Maven first and then re-think about separating client and server Enrico Il g

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

2018-10-16 Thread eolivelli
Github user eolivelli commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/670#discussion_r225419531 --- Diff: zookeeper-client/zookeeper-client-java/src/main/java/org/apache/zookeeper/client/ZKClientConfig.java --- @@ -23,7 +23,7 @@ import org.ap

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

2018-10-16 Thread nkalmar
Github user nkalmar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/670#discussion_r225420100 --- Diff: zookeeper-common/pom.xml --- @@ -0,0 +1,123 @@ + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instan

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

2018-10-16 Thread nkalmar
Github user nkalmar commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/670#discussion_r225420204 --- Diff: zookeeper-client/zookeeper-client-java/pom.xml --- @@ -0,0 +1,59 @@ + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/