Build failed in Jenkins: bookkeeper_release_branch_47_java8 #176

2018-10-03 Thread Apache Jenkins Server
See -- [...truncated 498.56 KB...] 2018-10-03T12:16:37.115 [INFO] Downloaded:

Build failed in Jenkins: bookkeeper_release_branch_47_integrationtests #164

2018-10-03 Thread Apache Jenkins Server
See -- [...truncated 477.31 KB...] 2018-10-03\T\13:02:53.735 [INFO] Downloading:

Jenkins build became unstable: bookkeeper_codecoverage #227

2018-10-03 Thread Apache Jenkins Server
See

[GitHub] eolivelli edited a comment on issue #1680: WIP - Drop 'stream' and 'streamTests' profiles

2018-10-03 Thread GitBox
eolivelli edited a comment on issue #1680: WIP - Drop 'stream' and 'streamTests' profiles URL: https://github.com/apache/bookkeeper/pull/1680#issuecomment-426602280 Okay, actually the real problem is with the Code coverage jobs, I would have to add additinal tricks on surefire

Build failed in Jenkins: bookkeeper_release_branch_48_integrationtests #49

2018-10-03 Thread Apache Jenkins Server
See -- Started by timer [EnvInject] - Loading node environment variables. Building remotely on H26 (ubuntu xenial) in workspace

[GitHub] eolivelli commented on issue #1680: WIP - Drop 'stream' and 'streamTests' profiles

2018-10-03 Thread GitBox
eolivelli commented on issue #1680: WIP - Drop 'stream' and 'streamTests' profiles URL: https://github.com/apache/bookkeeper/pull/1680#issuecomment-426602280 Okay, actally the real problem is with the Code coverage jobs, I would have to add additinal tricks on surefire configuration in

Build failed in Jenkins: bookkeeper_release_branch_47_java9 #176

2018-10-03 Thread Apache Jenkins Server
See -- [...truncated 504.52 KB...] 2018-10-03T12:59:45.297 [INFO] Downloaded:

Jenkins build is back to stable : bookkeeper_release_branch_46 #174

2018-10-03 Thread Apache Jenkins Server
See

Jenkins build is back to normal : bookkeeper_release_branch_48_java9 #49

2018-10-03 Thread Apache Jenkins Server
See

Jenkins build is back to normal : bookkeeper_release_nightly_snapshot #148

2018-10-03 Thread Apache Jenkins Server
See

[GitHub] ivankelly commented on issue #1712: Potential for TOAB violation with multiple calls to close

2018-10-03 Thread GitBox
ivankelly commented on issue #1712: Potential for TOAB violation with multiple calls to close URL: https://github.com/apache/bookkeeper/issues/1712#issuecomment-426593135 Demonstration of issue. https://github.com/ivankelly/bookkeeper/commit/2bcd8f32f176e618d0b214453eba84fe4a04f086

Jenkins build is still unstable: bookkeeper_postcommit_master_java9 #277

2018-10-03 Thread Apache Jenkins Server
See

[GitHub] sijie closed pull request #1724: [table service][cli] improve table service related cli commands

2018-10-03 Thread GitBox
sijie closed pull request #1724: [table service][cli] improve table service related cli commands URL: https://github.com/apache/bookkeeper/pull/1724 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] sijie closed pull request #1731: [table service][stats] enable stats for grpc calls on both client and server

2018-10-03 Thread GitBox
sijie closed pull request #1731: [table service][stats] enable stats for grpc calls on both client and server URL: https://github.com/apache/bookkeeper/pull/1731 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] eolivelli commented on issue #1734: [table service][client] provide scripts for uploading python clients to pypi

2018-10-03 Thread GitBox
eolivelli commented on issue #1734: [table service][client] provide scripts for uploading python clients to pypi URL: https://github.com/apache/bookkeeper/pull/1734#issuecomment-426805148 Do we need a formal 'release' process in order to publish this artifacts? I think so.

[GitHub] reddycharan commented on a change in pull request #1726: Introduce lifecycle components for managing components in AutoRecovery

2018-10-03 Thread GitBox
reddycharan commented on a change in pull request #1726: Introduce lifecycle components for managing components in AutoRecovery URL: https://github.com/apache/bookkeeper/pull/1726#discussion_r222465089 ## File path: bookkeeper-server/pom.xml ## @@ -112,6 +112,16 @@

[GitHub] reddycharan commented on a change in pull request #1726: Introduce lifecycle components for managing components in AutoRecovery

2018-10-03 Thread GitBox
reddycharan commented on a change in pull request #1726: Introduce lifecycle components for managing components in AutoRecovery URL: https://github.com/apache/bookkeeper/pull/1726#discussion_r222465123 ## File path: bookkeeper-server/pom.xml ## @@ -112,6 +112,16 @@

[GitHub] reddycharan commented on a change in pull request #1726: Introduce lifecycle components for managing components in AutoRecovery

2018-10-03 Thread GitBox
reddycharan commented on a change in pull request #1726: Introduce lifecycle components for managing components in AutoRecovery URL: https://github.com/apache/bookkeeper/pull/1726#discussion_r222465154 ## File path:

[GitHub] reddycharan commented on a change in pull request #1726: Introduce lifecycle components for managing components in AutoRecovery

2018-10-03 Thread GitBox
reddycharan commented on a change in pull request #1726: Introduce lifecycle components for managing components in AutoRecovery URL: https://github.com/apache/bookkeeper/pull/1726#discussion_r222465199 ## File path:

[GitHub] sijie commented on issue #1734: [table service][client] provide scripts for uploading python clients to pypi

2018-10-03 Thread GitBox
sijie commented on issue #1734: [table service][client] provide scripts for uploading python clients to pypi URL: https://github.com/apache/bookkeeper/pull/1734#issuecomment-426807268 @eolivelli yes this would be incorporated into formal release. currently the version is marked as

[GitHub] sijie opened a new pull request #1734: [table service][client] provide scripts for uploading python clients to pypi

2018-10-03 Thread GitBox
sijie opened a new pull request #1734: [table service][client] provide scripts for uploading python clients to pypi URL: https://github.com/apache/bookkeeper/pull/1734 Descriptions of the changes in this PR: *Motivation* make the python client available for usage

[GitHub] sijie commented on issue #1726: Introduce lifecycle components for managing components in AutoRecovery

2018-10-03 Thread GitBox
sijie commented on issue #1726: Introduce lifecycle components for managing components in AutoRecovery URL: https://github.com/apache/bookkeeper/pull/1726#issuecomment-426788917 run integration tests This is an automated

Build failed in Jenkins: bookkeeper_release_nightly_snapshot #147

2018-10-03 Thread Apache Jenkins Server
See Changes: [guosijie] Enhance Auditor -- [...truncated 5.59 MB...] Downloading:

Jenkins build is back to normal : bookkeeper-seed #331

2018-10-03 Thread Apache Jenkins Server
See

[GitHub] sijie commented on issue #1731: [table service][stats] enable stats for grpc calls on both client and server

2018-10-03 Thread GitBox
sijie commented on issue #1731: [table service][stats] enable stats for grpc calls on both client and server URL: https://github.com/apache/bookkeeper/pull/1731#issuecomment-426535842 rebuild java9 This is an automated

[GitHub] eolivelli closed pull request #1716: Fix Kerberos tests on JDK11:

2018-10-03 Thread GitBox
eolivelli closed pull request #1716: Fix Kerberos tests on JDK11: URL: https://github.com/apache/bookkeeper/pull/1716 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] eolivelli commented on issue #1680: WIP - Drop 'stream' and 'streamTests' profiles

2018-10-03 Thread GitBox
eolivelli commented on issue #1680: WIP - Drop 'stream' and 'streamTests' profiles URL: https://github.com/apache/bookkeeper/pull/1680#issuecomment-426527610 @sijie @jvrao news on this topic ? at the moment I think we should drop those profiles. We are distributing Table Service now

[GitHub] sijie commented on issue #1730: [table service][storage] improve the logic on creating the meta range for a table

2018-10-03 Thread GitBox
sijie commented on issue #1730: [table service][storage] improve the logic on creating the meta range for a table URL: https://github.com/apache/bookkeeper/pull/1730#issuecomment-426535547 run bookkeeper-server bookie tests run bookkeeper-server client tests

[GitHub] sijie commented on issue #1680: WIP - Drop 'stream' and 'streamTests' profiles

2018-10-03 Thread GitBox
sijie commented on issue #1680: WIP - Drop 'stream' and 'streamTests' profiles URL: https://github.com/apache/bookkeeper/pull/1680#issuecomment-426536490 > but it makes simpler the build and testing process, in particular CI jobs and code coverage jobs CI jobs and release process

[GitHub] eolivelli opened a new pull request #1732: Add Oracle JDK11 to Travis CI build

2018-10-03 Thread GitBox
eolivelli opened a new pull request #1732: Add Oracle JDK11 to Travis CI build URL: https://github.com/apache/bookkeeper/pull/1732 Descriptions of the changes in this PR: ### Motivation Start testing builds on JDK11, as it is the latest available version of Java ###

[GitHub] reddycharan commented on issue #1726: Introduce lifecycle components for managing components in AutoRecovery

2018-10-03 Thread GitBox
reddycharan commented on issue #1726: Introduce lifecycle components for managing components in AutoRecovery URL: https://github.com/apache/bookkeeper/pull/1726#issuecomment-426857495 @sijie i dont think this change is related to these test failures Test Result (9 failures / +9)

[GitHub] sijie commented on issue #1703: IllegalReferenceCountException at closing EntryLogManagerForSingleEntryLog

2018-10-03 Thread GitBox
] at io.netty.util.ReferenceCountUtil.safeRelease(ReferenceCountUtil.java:113) [netty-all-4.1.22.Final.jar:4.1.22.Final] at org.apache.bookkeeper.bookie.BufferedChannel.close(BufferedChannel.java:91) [bookkeeper-server-4.9.0-20181003-897c643-SNAPSHOT.jar:4.9.0-20181003-897c643-SNAPSHOT

[GitHub] sijie commented on issue #1726: Introduce lifecycle components for managing components in AutoRecovery

2018-10-03 Thread GitBox
sijie commented on issue #1726: Introduce lifecycle components for managing components in AutoRecovery URL: https://github.com/apache/bookkeeper/pull/1726#issuecomment-426880358 rebuild java8 rebuild java9 This is an

[GitHub] sijie commented on issue #1646: Use immutable metadata in LedgerHandle

2018-10-03 Thread GitBox
sijie commented on issue #1646: Use immutable metadata in LedgerHandle URL: https://github.com/apache/bookkeeper/pull/1646#issuecomment-426880529 @athanatos @jvrao can you review @ivankelly 's latest change? This is an

[GitHub] sijie closed issue #1660: Bookie can't start using 4.7.2 docker image

2018-10-03 Thread GitBox
sijie closed issue #1660: Bookie can't start using 4.7.2 docker image URL: https://github.com/apache/bookkeeper/issues/1660 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] sijie opened a new pull request #1735: Issue 1703: IllegalReferenceCountException at closing EntryLogManagerForSingleEntryLog

2018-10-03 Thread GitBox
sijie opened a new pull request #1735: Issue 1703: IllegalReferenceCountException at closing EntryLogManagerForSingleEntryLog URL: https://github.com/apache/bookkeeper/pull/1735 Descriptions of the changes in this PR: *Motivation* Fixes #1703. The active entry log

[GitHub] sijie commented on issue #1735: Issue 1703: IllegalReferenceCountException at closing EntryLogManagerForSingleEntryLog

2018-10-03 Thread GitBox
sijie commented on issue #1735: Issue 1703: IllegalReferenceCountException at closing EntryLogManagerForSingleEntryLog URL: https://github.com/apache/bookkeeper/pull/1735#issuecomment-426880012 @reddycharan I think this is related to multiple entry log changes. Can you take a look?

[GitHub] sijie commented on issue #1660: Bookie can't start using 4.7.2 docker image

2018-10-03 Thread GitBox
sijie commented on issue #1660: Bookie can't start using 4.7.2 docker image URL: https://github.com/apache/bookkeeper/issues/1660#issuecomment-426880087 This is fixed by #1666 This is an automated message from the Apache Git

[GitHub] codingwangqi commented on issue #1701: autorecovery-use-metadata-driver (part 4) : provide an `AuditorSelector` interface to abstract leader election logic for AuditorElector

2018-10-03 Thread GitBox
codingwangqi commented on issue #1701: autorecovery-use-metadata-driver (part 4) : provide an `AuditorSelector` interface to abstract leader election logic for AuditorElector URL: https://github.com/apache/bookkeeper/pull/1701#issuecomment-426881679 @sijie I addressed your comment. please

[GitHub] codingwangqi commented on a change in pull request #1701: autorecovery-use-metadata-driver (part 4) : provide an `AuditorSelector` interface to abstract leader election logic for AuditorElect

2018-10-03 Thread GitBox
codingwangqi commented on a change in pull request #1701: autorecovery-use-metadata-driver (part 4) : provide an `AuditorSelector` interface to abstract leader election logic for AuditorElector URL: https://github.com/apache/bookkeeper/pull/1701#discussion_r222534734 ## File path:

[GitHub] sijie closed pull request #1730: [table service][storage] improve the logic on creating the meta range for a table

2018-10-03 Thread GitBox
sijie closed pull request #1730: [table service][storage] improve the logic on creating the meta range for a table URL: https://github.com/apache/bookkeeper/pull/1730 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] sijie opened a new pull request #1733: [table service][client] storage client can open tables on a different namespace

2018-10-03 Thread GitBox
sijie opened a new pull request #1733: [table service][client] storage client can open tables on a different namespace URL: https://github.com/apache/bookkeeper/pull/1733 Descriptions of the changes in this PR: *Motivation* In some cases, we are using same storage client

[GitHub] eolivelli closed pull request #1732: Add Oracle JDK11 to Travis CI build

2018-10-03 Thread GitBox
eolivelli closed pull request #1732: Add Oracle JDK11 to Travis CI build URL: https://github.com/apache/bookkeeper/pull/1732 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] sijie commented on issue #1726: Introduce lifecycle components for managing components in AutoRecovery

2018-10-03 Thread GitBox
sijie commented on issue #1726: Introduce lifecycle components for managing components in AutoRecovery URL: https://github.com/apache/bookkeeper/pull/1726#issuecomment-426562447 run integration tests This is an automated

[GitHub] ivankelly commented on issue #1646: Use immutable metadata in LedgerHandle

2018-10-03 Thread GitBox
ivankelly commented on issue #1646: Use immutable metadata in LedgerHandle URL: https://github.com/apache/bookkeeper/pull/1646#issuecomment-426551590 rerun bookkeeper-server bookie tests This is an automated message from the

[GitHub] eolivelli commented on issue #1732: Add Oracle JDK11 to Travis CI build

2018-10-03 Thread GitBox
eolivelli commented on issue #1732: Add Oracle JDK11 to Travis CI build URL: https://github.com/apache/bookkeeper/pull/1732#issuecomment-426554859 IGNORE CI This is an automated message from the Apache Git Service. To respond

[GitHub] eolivelli commented on issue #1732: Add Oracle JDK11 to Travis CI build

2018-10-03 Thread GitBox
eolivelli commented on issue #1732: Add Oracle JDK11 to Travis CI build URL: https://github.com/apache/bookkeeper/pull/1732#issuecomment-426554834 Travis is OK continuous-integration/travis-ci/pr this change is not related to other jobs I will ignore CI failures and commit

[GitHub] sijie commented on issue #1733: [table service][client] storage client can open tables on a different namespace

2018-10-03 Thread GitBox
sijie commented on issue #1733: [table service][client] storage client can open tables on a different namespace URL: https://github.com/apache/bookkeeper/pull/1733#issuecomment-426562305 run pr validation This is an