[GitHub] jiazhai commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-20 Thread git
jiazhai commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-316894211 @caiok, Thanks for the explain. It is better to handle the code duplications in dirs 4.4.0 and 4.4.0-alpine. If it is hard t

[GitHub] jiazhai commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image

2017-07-20 Thread git
jiazhai commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-316894211 @caiok, Thanks for the explain. It is better to handle the code duplications in dirs 4.4.0 and 4.4.0-alpine. If it is hard t

[GitHub] jiazhai closed pull request #264: Issue 263: Enable request rate limiter in zookeeper client

2017-07-20 Thread git
jiazhai closed pull request #264: Issue 263: Enable request rate limiter in zookeeper client URL: https://github.com/apache/bookkeeper/pull/264 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] jiazhai closed issue #263: Enable request rate limiter in zookeeper client

2017-07-20 Thread git
jiazhai closed issue #263: Enable request rate limiter in zookeeper client URL: https://github.com/apache/bookkeeper/issues/263 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] jiazhai closed pull request #268: Issue 267: Bookie should serve in read only mode during shutting down.

2017-07-20 Thread git
jiazhai closed pull request #268: Issue 267: Bookie should serve in read only mode during shutting down. URL: https://github.com/apache/bookkeeper/pull/268 This is an automated message from the Apache Git Service. To respon

[GitHub] jiazhai closed issue #267: Turn Bookie to read only mode during shutting down.

2017-07-20 Thread git
jiazhai closed issue #267: Turn Bookie to read only mode during shutting down. URL: https://github.com/apache/bookkeeper/issues/267 This is an automated message from the Apache Git Service. To respond to the message, please

Re: New BookKeeper homepage on GitHub

2017-07-20 Thread Jia Zhai
👍, Thanks Sijie and Enrico for this. :) On Thu, Jul 20, 2017 at 9:45 PM, Enrico Olivelli wrote: > Hi, > we have a new "homepage" on GitHub !!! > > see > https://github.com/apache/bookkeeper/ > > > Enrico >

Build failed in Jenkins: bookkeeper-master #1824

2017-07-20 Thread Apache Jenkins Server
See -- [...truncated 33.53 KB...] Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 11.937 sec - in org.apache.bookkeeper.client.TestLedgerFragmentReplication Running org.

Re: Working on CI

2017-07-20 Thread Enrico Olivelli
I apologize I will do Enrico Il gio 20 lug 2017, 18:07 Sijie Guo ha scritto: > You might set the email to your personal email when you are setting up a > new CI job, so it can reduce the noisy emails to dev@. > > Sijie > > On Jul 20, 2017 6:55 AM, "Enrico Olivelli" wrote: > > > I am currently w

Re: Working on CI

2017-07-20 Thread Sijie Guo
You might set the email to your personal email when you are setting up a new CI job, so it can reduce the noisy emails to dev@. Sijie On Jul 20, 2017 6:55 AM, "Enrico Olivelli" wrote: > I am currently working on Jenkins on bookkeeper-master and a new > bookkeeper-tests job. > Do not worry about

[GitHub] eolivelli opened a new issue #270: Provide LedgerHandlerAdv#asyncAddEntry which takes a ByteBuf

2017-07-20 Thread git
eolivelli opened a new issue #270: Provide LedgerHandlerAdv#asyncAddEntry which takes a ByteBuf URL: https://github.com/apache/bookkeeper/issues/270 in 4.5 we introduced addEntry overloaded methods which take Netty 4 ByteBufs. A similar API is missing on LedgerHandlerAdv. We should ad

Re: BookKeeper and code-coverage - JaCoco

2017-07-20 Thread Enrico Olivelli
Jacoco is running as expected see: https://coveralls.io/jobs/27785961 I think it is better to switch to Jacoco I wonder why on DL there is no such problem with Cobertura @Sijie do you have any idea ? -- Enrico 2017-07-20 16:06 GMT+02:00 Enrico Olivelli : > I am sorry but it seems that cober

[jira] [Commented] (BOOKKEEPER-1027) Cleanup main README and main website page

2017-07-20 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/BOOKKEEPER-1027?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16094791#comment-16094791 ] Hudson commented on BOOKKEEPER-1027: FAILURE: Integrated in Jenkins build bookkee

Build failed in Jenkins: bookkeeper-master #1823

2017-07-20 Thread Apache Jenkins Server
See Changes: [eolivelli] BOOKKEEPER-1027: Cleanup README -- [...truncated 34.97 KB...] Running org.apache.bookkeeper.client.LedgerCloseTest Tests run: 4, Failures: 0, Err

Re: BookKeeper and code-coverage - JaCoco

2017-07-20 Thread Enrico Olivelli
I am sorry but it seems that cobertura is not working with java8, as reported in Coveralls.io homepage https://github.com/trautonen/coveralls-maven-plugin see our error: https://builds.apache.org/job/bookkeeper-tests/6/console I will be back to Jacoco which seems to be the only one which support

Build failed in Jenkins: bookkeeper-tests #5

2017-07-20 Thread Apache Jenkins Server
See -- Started by user eolivelli [EnvInject] - Loading node environment variables. Building remotely on H9 (Hadoop) in workspace > git

Working on CI

2017-07-20 Thread Enrico Olivelli
I am currently working on Jenkins on bookkeeper-master and a new bookkeeper-tests job. Do not worry about emails about failed builds I will be back once the configuration is OK Enrico

Build failed in Jenkins: bookkeeper-tests #2

2017-07-20 Thread Apache Jenkins Server
See -- [...truncated 542.44 KB...] [INFO] [INFO] --- maven-resources-plugin:2.7:testResources (default-testResources) @ bookkeeper-stats-api --- [INFO] Using 'UTF-8' encoding to copy filte

Build failed in Jenkins: bookkeeper-tests #1

2017-07-20 Thread Apache Jenkins Server
See -- [...truncated 42.06 KB...] [INFO] Data file does not contain instrumentation information for the file org/apache/bookkeeper/stats/StatsLogger.java. Ensure this class was instrument

New BookKeeper homepage on GitHub

2017-07-20 Thread Enrico Olivelli
Hi, we have a new "homepage" on GitHub !!! see https://github.com/apache/bookkeeper/ Enrico

Custom ZookKeeper configuration

2017-07-20 Thread Enrico Olivelli
Hi, I would like to enhance the BookKeeper#Builder API order to let the user pass the ZKClientConfiig. I think it is a new ZK 3.5 class. In general we should refactor that API because it deals directly with Netty 4 classes and I would like a more future-proof API. I can start a BP for this kind

[GitHub] eolivelli closed pull request #248: BOOKKEEPER-1027: Cleanup README

2017-07-20 Thread git
eolivelli closed pull request #248: BOOKKEEPER-1027: Cleanup README URL: https://github.com/apache/bookkeeper/pull/248 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] eolivelli commented on issue #264: Issue 263: Enable request rate limiter in zookeeper client

2017-07-20 Thread git
eolivelli commented on issue #264: Issue 263: Enable request rate limiter in zookeeper client URL: https://github.com/apache/bookkeeper/pull/264#issuecomment-316700642 @sijie can we use the same configuration parameter in BookKeeper constructor ? I can file another issue

[GitHub] eolivelli commented on issue #185: BOOKKEEPER-1017 Create documentation for ZooKeeper ACLs

2017-07-20 Thread git
eolivelli commented on issue #185: BOOKKEEPER-1017 Create documentation for ZooKeeper ACLs URL: https://github.com/apache/bookkeeper/pull/185#issuecomment-316691350 @jiazhai is this OK for you now ? This is an automated mes

[GitHub] eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-20 Thread git
eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128501855 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0

[GitHub] eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-20 Thread git
eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128501217 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0

[GitHub] eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-20 Thread git
eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128501585 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0

[GitHub] eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-20 Thread git
eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128501659 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0

[GitHub] eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-20 Thread git
eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128501335 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0

[GitHub] eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-20 Thread git
eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128501094 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0

[GitHub] eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-20 Thread git
eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128502713 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieInitializationTest.java

[GitHub] eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-20 Thread git
eolivelli commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128502522 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieInitializationTest.java

[GitHub] eolivelli opened a new issue #269: Upgrade findbugs to 3.0.4

2017-07-20 Thread git
eolivelli opened a new issue #269: Upgrade findbugs to 3.0.4 URL: https://github.com/apache/bookkeeper/issues/269 We are using 3.0.3, let's upgrade to 3.0.4 This is an automated message from the Apache Git Service. To respond

Re: BookKeeper 4.5.0 performance regression ?

2017-07-20 Thread Enrico Olivelli
Kishore, do you have news? Il ven 14 lug 2017, 09:05 Enrico Olivelli ha scritto: > At the meeting we told the Kishore will perform some benchmarks on his > side. > He will take a look at my code, and we are going to share the results. > Maybe it will be possible to share the results of benchmark

Using ZooKeeper Netty connector (ClientCnxnSocketNetty)

2017-07-20 Thread Enrico Olivelli
Hi, I am playing with ZooKeeper configuration and ClientCnxnSocketNetty vs ClientCnxnSocketNIO. The ClientCnxnSocketNetty is an implementation of the network layer of ZooKeeper client which uses Netty 3. I have asked on the u...@zookeeper.apache.org list about which are the potenzial benefits. Do