[GitHub] eolivelli commented on issue #1340: Only check ipv6 binding when /sbin/sysctl exists

2018-04-13 Thread GitBox
eolivelli commented on issue #1340: Only check ipv6 binding when /sbin/sysctl exists URL: https://github.com/apache/bookkeeper/pull/1340#issuecomment-381306191 It tests failed This is an automated message from the Apache Git

[GitHub] eolivelli commented on issue #1342: Provide memory & gc options in bookkeeper script

2018-04-13 Thread GitBox
eolivelli commented on issue #1342: Provide memory & gc options in bookkeeper script URL: https://github.com/apache/bookkeeper/pull/1342#issuecomment-381305263 Looks good. As you are working on this can you take a look at adding JAVA_JDK_OPT and add jdk internal module 'opens' to

[GitHub] reddycharan commented on issue #1343: Advisory (optimistic) write close

2018-04-13 Thread GitBox
reddycharan commented on issue #1343: Advisory (optimistic) write close URL: https://github.com/apache/bookkeeper/issues/1343#issuecomment-381291042 @jvrao @sijie fyi This is an automated message from the Apache Git Service.

[GitHub] reddycharan opened a new issue #1343: Advisory (optimistic) write close

2018-04-13 Thread GitBox
reddycharan opened a new issue #1343: Advisory (optimistic) write close URL: https://github.com/apache/bookkeeper/issues/1343 **FEATURE REQUEST** 1. Please describe the feature you are requesting. With https://github.com/apache/bookkeeper/issues/570 entrylog per ledger

[GitHub] sijie opened a new pull request #1342: Provide memory & gc options in bookkeeper script

2018-04-13 Thread GitBox
sijie opened a new pull request #1342: Provide memory & gc options in bookkeeper script URL: https://github.com/apache/bookkeeper/pull/1342 Descriptions of the changes in this PR: It is convenient to provide `MEM_OPTS` and `GC_OPTS` in bookkeeper script and bkenv.sh to allow

[GitHub] sijie opened a new pull request #1341: Update release schedule for 4.8.0

2018-04-13 Thread GitBox
sijie opened a new pull request #1341: Update release schedule for 4.8.0 URL: https://github.com/apache/bookkeeper/pull/1341 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 #1340: Only check ipv6 binding when /sbin/sysctl exists

2018-04-13 Thread GitBox
sijie opened a new pull request #1340: Only check ipv6 binding when /sbin/sysctl exists URL: https://github.com/apache/bookkeeper/pull/1340 Descriptions of the changes in this PR: *Problem* If bookkeeper is included in some slim docker images, sysctl might not exists at all.

[GitHub] sijie closed pull request #1338: Fallback to use `getZkServers` and `getZkLedgersPath` when resolving from metadata service uri

2018-04-13 Thread GitBox
sijie closed pull request #1338: Fallback to use `getZkServers` and `getZkLedgersPath` when resolving from metadata service uri URL: https://github.com/apache/bookkeeper/pull/1338 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] sijie opened a new pull request #1338: Fallback to use `getZkServers` and `getZkLedgersPath` when resolving from metadata service uri

2018-04-13 Thread GitBox
sijie opened a new pull request #1338: Fallback to use `getZkServers` and `getZkLedgersPath` when resolving from metadata service uri URL: https://github.com/apache/bookkeeper/pull/1338 Descriptions of the changes in this PR: *Problme* Dlog tests are failing because dlog is