[GitHub] [zookeeper] JiriOndrusek edited a comment on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
JiriOndrusek edited a comment on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492933860 @eolivelli Ok, i'll continue with osgi packaging. I'll apply this also to

[GitHub] [zookeeper] JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492933860 @eolivelli Ok, i'll continue with osgi packaging. I'll apply this also to zookeep

ZooKeeper_branch34_jdk7 - Build # 2328 - Failure

2019-05-15 Thread Apache Jenkins Server
See https://builds.apache.org/job/ZooKeeper_branch34_jdk7/2328/ ### ## LAST 60 LINES OF THE CONSOLE ### [...truncated 46.89 KB...] [junit] Running org.apache.zookeep

[GitHub] [zookeeper] anmolnar commented on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size

2019-05-15 Thread GitBox
anmolnar commented on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size URL: https://github.com/apache/zookeeper/pull/770#issuecomment-492824563 @enixon Thanks, it's great you already have a fix in mind. No worries and no rush, just take your time. ---

[GitHub] [zookeeper] anmolnar edited a comment on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size

2019-05-15 Thread GitBox
anmolnar edited a comment on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size URL: https://github.com/apache/zookeeper/pull/770#issuecomment-492824563 @enixon Thanks, it's great you already have a fix. No worries and no rush, just take your time.

[GitHub] [zookeeper] eolivelli commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
eolivelli commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492824404 It would be great to have some integration test which loads zookeeper client with osgi a

[GitHub] [zookeeper] enixon commented on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size

2019-05-15 Thread GitBox
enixon commented on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size URL: https://github.com/apache/zookeeper/pull/770#issuecomment-492812456 Created a new ticket (3396) and opened #949 to resolve (cc @anmolnar - sorry for the trouble!)

[jira] [Updated] (ZOOKEEPER-3396) Flaky test in RestoreCommittedLogTest

2019-05-15 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3396?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] ASF GitHub Bot updated ZOOKEEPER-3396: -- Labels: pull-request-available (was: ) > Flaky test in RestoreCommittedLogTest > -

[jira] [Created] (ZOOKEEPER-3396) Flaky test in RestoreCommittedLogTest

2019-05-15 Thread Brian Nixon (JIRA)
Brian Nixon created ZOOKEEPER-3396: -- Summary: Flaky test in RestoreCommittedLogTest Key: ZOOKEEPER-3396 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3396 Project: ZooKeeper Issue Typ

[GitHub] [zookeeper] enixon opened a new pull request #949: ZOOKEEPER-3396: Flaky test in RestoreCommittedLogTest

2019-05-15 Thread GitBox
enixon opened a new pull request #949: ZOOKEEPER-3396: Flaky test in RestoreCommittedLogTest URL: https://github.com/apache/zookeeper/pull/949 Adjusting parameters of RestoreCommittedLogTest to avoid failures caused by threshold configured to be below the minimum allowed file size ---

[GitHub] [zookeeper] enixon commented on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size

2019-05-15 Thread GitBox
enixon commented on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size URL: https://github.com/apache/zookeeper/pull/770#issuecomment-492804816 @anmolnar It looks like the issue is FileTxnLog::getCurrentLogSize returns a multiple of a high number that is dependent on the

[GitHub] [zookeeper] JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492796520 Hi @nkalmar @eolivelli, let me introduce "a better solution" for the problem wi

[GitHub] [zookeeper] enixon commented on a change in pull request #899: ZOOKEEPER-3354: Improve efficiency of DeleteAllCommand

2019-05-15 Thread GitBox
enixon commented on a change in pull request #899: ZOOKEEPER-3354: Improve efficiency of DeleteAllCommand URL: https://github.com/apache/zookeeper/pull/899#discussion_r284359820 ## File path: zookeeper-server/src/main/java/org/apache/zookeeper/ZKUtil.java ## @@ -45,20 +48,

[jira] [Assigned] (ZOOKEEPER-3263) Illegal reflective access in zookeer's kerberosUtil

2019-05-15 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andor Molnar reassigned ZOOKEEPER-3263: --- Assignee: Andor Molnar > Illegal reflective access in zookeer's kerberosUtil > -

[jira] [Commented] (ZOOKEEPER-3263) Illegal reflective access in zookeer's kerberosUtil

2019-05-15 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16840556#comment-16840556 ] Andor Molnar commented on ZOOKEEPER-3263: - Issues that I've found so far: {no

[jira] [Commented] (ZOOKEEPER-3263) Illegal reflective access in zookeer's kerberosUtil

2019-05-15 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16840544#comment-16840544 ] Andor Molnar commented on ZOOKEEPER-3263: - [~bansalp] Would you please confir

[jira] [Updated] (ZOOKEEPER-3263) Illegal reflective access in zookeer's kerberosUtil

2019-05-15 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andor Molnar updated ZOOKEEPER-3263: Affects Version/s: 3.4.13 > Illegal reflective access in zookeer's kerberosUtil > -

[GitHub] [zookeeper] eolivelli commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
eolivelli commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492714723 Maybe you can simply add raw entries to the manifest in the jar plugin configuration --

[GitHub] [zookeeper] JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492704512 @nkalmar @eolivelli I'll continue with this tomorrow. I see in your comment: "T

[jira] [Commented] (ZOOKEEPER-3353) Admin commands for showing initial settings

2019-05-15 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3353?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16840477#comment-16840477 ] Hudson commented on ZOOKEEPER-3353: --- SUCCESS: Integrated in Jenkins build ZooKeeper

[GitHub] [zookeeper] eolivelli commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
eolivelli commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492683153 Yes changing the package name is feasible but only for 3.6+. ---

[GitHub] [zookeeper] nkalmar commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
nkalmar commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492680330 I see you changed zookeeper-server packaging from jar to bundle. Yeah... I will check up o

[GitHub] [zookeeper] nkalmar edited a comment on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
nkalmar edited a comment on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492677907 There will be 2 jars upon release, zookeeper-jute and zookeeper-server. zookeeper-j

[GitHub] [zookeeper] nkalmar commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
nkalmar commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492677907 There will be 2 jars upon release, zookeeper-jute and zookeeper-server. zookeeper-jute is

[GitHub] [zookeeper] JiriOndrusek edited a comment on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
JiriOndrusek edited a comment on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492672683 @nkalmar @eolivelli I'm getting close to the finishing. I just want to clari

[GitHub] [zookeeper] JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492672683 @nkalmar @eolivelli I'm getting close to the finishing. I just want to clarify some

[GitHub] [zookeeper] anmolnar commented on a change in pull request #899: ZOOKEEPER-3354: Improve efficiency of DeleteAllCommand

2019-05-15 Thread GitBox
anmolnar commented on a change in pull request #899: ZOOKEEPER-3354: Improve efficiency of DeleteAllCommand URL: https://github.com/apache/zookeeper/pull/899#discussion_r284260180 ## File path: zookeeper-server/src/main/java/org/apache/zookeeper/ZKUtil.java ## @@ -45,20 +4

[jira] [Commented] (ZOOKEEPER-3353) Admin commands for showing initial settings

2019-05-15 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3353?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16840369#comment-16840369 ] Hudson commented on ZOOKEEPER-3353: --- SUCCESS: Integrated in Jenkins build Zookeeper

[GitHub] [zookeeper] anmolnar commented on issue #898: ZOOKEEPER-3353: Admin commands for showing initial settings

2019-05-15 Thread GitBox
anmolnar commented on issue #898: ZOOKEEPER-3353: Admin commands for showing initial settings URL: https://github.com/apache/zookeeper/pull/898#issuecomment-492604226 Committed to master branch. Thanks @enixon ! This is an au

[GitHub] [zookeeper] anmolnar commented on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size

2019-05-15 Thread GitBox
anmolnar commented on issue #770: ZOOKEEPER-3244: Add option to snapshot based on log size URL: https://github.com/apache/zookeeper/pull/770#issuecomment-492603934 @enixon Please double check your test code, it looks like this patch has introduced a flaky test: https://builds.apache.or

[jira] [Assigned] (ZOOKEEPER-3353) Admin commands for showing initial settings

2019-05-15 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3353?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andor Molnar reassigned ZOOKEEPER-3353: --- Assignee: Brian Nixon > Admin commands for showing initial settings > --

[jira] [Resolved] (ZOOKEEPER-3353) Admin commands for showing initial settings

2019-05-15 Thread Andor Molnar (JIRA)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-3353?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andor Molnar resolved ZOOKEEPER-3353. - Resolution: Fixed Fix Version/s: 3.6.0 Issue resolved by pull request 898 [htt

[GitHub] [zookeeper] asfgit closed pull request #898: ZOOKEEPER-3353: Admin commands for showing initial settings

2019-05-15 Thread GitBox
asfgit closed pull request #898: ZOOKEEPER-3353: Admin commands for showing initial settings URL: https://github.com/apache/zookeeper/pull/898 This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [zookeeper] nkalmar commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
nkalmar commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492587354 I agree with Enrico, please just continue on this branch, as maven will be used to release

[GitHub] [zookeeper] eolivelli commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
eolivelli commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492569164 @JiriOndrusek I feel you can continue on this branch, we can merge all this OSGI stuff i

[GitHub] [zookeeper] JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator)

2019-05-15 Thread GitBox
JiriOndrusek commented on issue #945: [ZOOKEEPER-3389] Zookeeper does not export all required packages in OSGi (needed for curator) URL: https://github.com/apache/zookeeper/pull/945#issuecomment-492561609 Hi @nkalmar , I've started with osgification of maven build, so far in my branch h