Draft Agenda for Bi-weekly meeting at 2017-07-27

Hi all, :) I would like to put a draft Agenda here for 7/27's meeting, this is just limited from my point of view, Please feel free to add anything you want to discuss in this meeting. *Agenda:* - Status of 4.5

Re: BP-11: New BookKeeper Website

Awesome On Wed, Jul 26, 2017 at 6:04 AM, Ivan Kelly wrote: > Looks great :D. This has been long overdue. > > My one piece of feedback is that perhaps the font is too big on the > documentation. It means that there isn't much content on the page > before having to scroll (See

Re: [DISCUSS] Slack Channel for BookKeeper

Currently BK doesn't has one yet. On Tue, Jul 25, 2017 at 9:29 AM, Henry Saputra wrote: > Is Apache BookKeeper has its own Slack team? If it has then we could just > add a channel for DistLog. > > - Henry > > On Mon, Jul 24, 2017 at 11:42 PM, Enrico Olivelli

[GitHub] zhaijack opened a new pull request #298: Issue 295: LocalBookKeeper fails to start up

zhaijack opened a new pull request #298: Issue 295: LocalBookKeeper fails to start up URL: https://github.com/apache/bookkeeper/pull/298 Descriptions of the changes in this PR: remote the zk port and zk servers from the classes and use the values passed in from the method.

[GitHub] zhaijack commented on issue #297: Issue 296: Bookie supports ephemeral port

zhaijack commented on issue #297: Issue 296: Bookie supports ephemeral port URL: https://github.com/apache/bookkeeper/pull/297#issuecomment-317907200 ported the change for @sijie This is an automated message from the Apache

[GitHub] zhaijack opened a new pull request #297: Issue 296: Bookie supports ephemeral port

zhaijack opened a new pull request #297: Issue 296: Bookie supports ephemeral port URL: https://github.com/apache/bookkeeper/pull/297 Descriptions of the changes in this PR: - add a flag to allow/disable using ephemeral ports - change the initialization sequence to support

Re: BP-11: New BookKeeper Website

Looks great :D. This has been long overdue. My one piece of feedback is that perhaps the font is too big on the documentation. It means that there isn't much content on the page before having to scroll (See attached screenshot). My resolution is 1360x768. -Ivan On Tue, Jul 25, 2017 at 9:38 PM,

Re: [DISCUSS] Slack Channel for BookKeeper

On Tue, Jul 25, 2017 at 2:57 PM, Ivan Kelly wrote: > Is there an apache "team" in slack? it would save having to have > multiple teams open if a person is involved in multiple projects. This > would be especially useful for people who use slack in the browser. It > would also

Re: [DISCUSS] Slack Channel for BookKeeper

Is there an apache "team" in slack? it would save having to have multiple teams open if a person is involved in multiple projects. This would be especially useful for people who use slack in the browser. It would also reduce the signup barrier. -Ivan On Tue, Jul 25, 2017 at 6:29 PM, Henry

[GitHub] zhaijack opened a new issue #296: Bookies supports ephemeral port

zhaijack opened a new issue #296: Bookies supports ephemeral port URL: https://github.com/apache/bookkeeper/issues/296 **FEATURE REQUEST** 1. Please describe the feature you are requesting. // If the caller specified ephemeral ports then use ephemeral ports for

[GitHub] zhaijack commented on issue #295: LocalBookKeeper failed to start

zhaijack commented on issue #295: LocalBookKeeper failed to start URL: https://github.com/apache/bookkeeper/issues/295#issuecomment-317873950 this should be a **blocker** for 4.5.0 This is an automated message from the

[GitHub] zhaijack opened a new issue #295: LocalBookKeeper failed to start

zhaijack opened a new issue #295: LocalBookKeeper failed to start URL: https://github.com/apache/bookkeeper/issues/295 **BUG REPORT** 1. Please describe the issue you observed: - What did you do? Upgrade BK version to 4.5.0 in DL and run the tests. - What did you

Re: BP-11: New BookKeeper Website

Awesome !! lets make it dense. :) On Tue, Jul 25, 2017 at 12:15 PM, Yiming Zang wrote: > This looks awesome! > > On Tue, Jul 25, 2017 at 12:09 PM, Sijie Guo wrote: > > > Hi all, > > > > Luc Perkins is helping on a new bookkeeper documentation site

Re: BP-11: New BookKeeper Website

This looks awesome! On Tue, Jul 25, 2017 at 12:09 PM, Sijie Guo wrote: > Hi all, > > Luc Perkins is helping on a new bookkeeper documentation site for BP-11. > Although it is still WIP, we'd like to first share the beta version with > the community to get some feedbacks. > >

[GitHub] yzang commented on issue #266: Issue 265: Add persistable bookie status

yzang commented on issue #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#issuecomment-317842271 @sijie That make sense, I understand what you mean, but my concern is: Even if we persist the bookie status on disk, when we set the

BP-11: New BookKeeper Website

Hi all, Luc Perkins is helping on a new bookkeeper documentation site for BP-11. Although it is still WIP, we'd like to first share the beta version with the community to get some feedbacks. The new website: https://lucperkins.github.io/bookkeeper/ The pull request (still WIP) :

[GitHub] sijie commented on issue #294: BP-11: New BooKeeper Website

sijie commented on issue #294: BP-11: New BooKeeper Website URL: https://github.com/apache/bookkeeper/issues/294#issuecomment-317837421 ^ @lucperkins This is an automated message from the Apache Git Service. To respond to

[GitHub] sijie opened a new issue #294: BP-11: New BooKeeper Website

sijie opened a new issue #294: BP-11: New BooKeeper Website URL: https://github.com/apache/bookkeeper/issues/294 **FEATURE REQUEST** 1. Please describe the feature you are requesting. - new bookkeeper website/documentation (move the build from Apache CMS to Jekyll)

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

sijie commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-317831799 @caiok Let me try to summarize here. There are two approaches: 1. the approach suggested in this pull request. a

[GitHub] lucperkins opened a new pull request #293: Beta version of the new website (work in progress)

lucperkins opened a new pull request #293: Beta version of the new website (work in progress) URL: https://github.com/apache/bookkeeper/pull/293 This is an automated message from the Apache Git Service. To respond to the

Re: EnsemblePlacementPolicy Public API breaking changes

Il mar 25 lug 2017, 19:08 Sijie Guo ha scritto: > On Tue, Jul 25, 2017 at 9:24 AM, Enrico Olivelli > wrote: > > > Il mar 25 lug 2017, 18:12 Sijie Guo ha scritto: > > > > > Let me try to summarize the comments here (also as a

Re: EnsemblePlacementPolicy Public API breaking changes

On Tue, Jul 25, 2017 at 9:24 AM, Enrico Olivelli wrote: > Il mar 25 lug 2017, 18:12 Sijie Guo ha scritto: > > > Let me try to summarize the comments here (also as a reference for other > > people). This basically is comprised of two questions: > > > > 1)

[GitHub] yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129364263 ## File path:

[GitHub] yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129363913 ## File path: bookkeeper-server/pom.xml ## @@ -217,11 +217,21 @@ ${netty.version}

[GitHub] yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

yzang commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129363712 ## File path: bookkeeper-server/pom.xml ## @@ -217,11 +217,21 @@ ${netty.version}

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

caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-317797541 @sijie Thanks for the reply. We don't push images on dockerhub, dockerhub checkout our dockerfiles and builds them. Check [this

Re: [DISCUSS] Slack Channel for BookKeeper

Is Apache BookKeeper has its own Slack team? If it has then we could just add a channel for DistLog. - Henry On Mon, Jul 24, 2017 at 11:42 PM, Enrico Olivelli wrote: > Awesome +1 > > Il mar 25 lug 2017, 05:39 Dustin Castor > ha > scritto: >

Re: EnsemblePlacementPolicy Public API breaking changes

Il mar 25 lug 2017, 18:12 Sijie Guo ha scritto: > Let me try to summarize the comments here (also as a reference for other > people). This basically is comprised of two questions: > > 1) how do we handle the 'binary backward compatibility' for drop-in upgrade > in

[GitHub] sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

sijie commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129353429 ## File path:

[GitHub] sijie commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

sijie commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#issuecomment-317789926 >> "I wonder if we really need to implement so many variants of the http server or simply implement one" different

Re: EnsemblePlacementPolicy Public API breaking changes

Let me try to summarize the comments here (also as a reference for other people). This basically is comprised of two questions: 1) how do we handle the 'binary backward compatibility' for drop-in upgrade in applications using only one bk version? ideally if we change any method signatures in

[GitHub] eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs

eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs URL: https://github.com/apache/bookkeeper/pull/276#discussion_r129337447 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntry.java ##

[GitHub] sijie commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs

sijie commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs URL: https://github.com/apache/bookkeeper/pull/276#discussion_r129336829 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntry.java ## @@

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

sijie commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-317770478 @caiok : regarding the versioning here, I am not sure whether we need to keep the version here. Ideally the docker image should

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129330400 ## File path: bookkeeper-server/pom.xml ## @@ -217,11 +217,21 @@ ${netty.version}

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129329168 ## File path:

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129329619 ## File path:

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129330300 ## File path:

[GitHub] eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

eolivelli commented on a change in pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#discussion_r129331345 ## File path: bookkeeper-server/pom.xml ## @@ -217,11 +217,21 @@ ${netty.version}

[GitHub] sijie commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs

sijie commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs URL: https://github.com/apache/bookkeeper/pull/276#discussion_r129331061 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntry.java ## @@

[GitHub] eolivelli commented on issue #290: Add checkstyle:checkstyle to bk-merge script

eolivelli commented on issue #290: Add checkstyle:checkstyle to bk-merge script URL: https://github.com/apache/bookkeeper/issues/290#issuecomment-317762194 closing as won't fix. checkstyle is already bound to "validate" phase for every project which supports it, it is not active in

[GitHub] eolivelli closed issue #290: Add checkstyle:checkstyle to bk-merge script

eolivelli closed issue #290: Add checkstyle:checkstyle to bk-merge script URL: https://github.com/apache/bookkeeper/issues/290 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] eolivelli commented on issue #291: Issue-290 Add checkstyle:checkstyle to bk-merge script

eolivelli commented on issue #291: Issue-290 Add checkstyle:checkstyle to bk-merge script URL: https://github.com/apache/bookkeeper/pull/291#issuecomment-317761995 @sijie sure, sorry I missed it. Usually I am working with bookkeeper-server, I forgot about the fact we have only in minor

[GitHub] eolivelli closed pull request #291: Issue-290 Add checkstyle:checkstyle to bk-merge script

eolivelli closed pull request #291: Issue-290 Add checkstyle:checkstyle to bk-merge script URL: https://github.com/apache/bookkeeper/pull/291 This is an automated message from the Apache Git Service. To respond to the

Re: Bookie Http Endpoint

Dustin, Enrico, If you guys have time, please take a look at Yiming's new pull request : https://github.com/apache/bookkeeper/pull/278 - Sijie On Wed, Jul 19, 2017 at 9:46 PM, Enrico Olivelli wrote: > Il gio 20 lug 2017, 00:37 Sijie Guo ha scritto: >

[GitHub] sijie commented on issue #291: Issue-290 Add checkstyle:checkstyle to bk-merge script

sijie commented on issue #291: Issue-290 Add checkstyle:checkstyle to bk-merge script URL: https://github.com/apache/bookkeeper/pull/291#issuecomment-317760987 Why do you need this? 'checkstyle' is in already in `validate` phase.

[GitHub] sijie commented on issue #266: Issue 265: Add persistable bookie status

sijie commented on issue #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#issuecomment-317760092 @yzang : I think the ideal override sequence of write/readonly status would be: 1. if a bookie is started with '--readonly' option, it

[GitHub] caiok opened a new issue #292: Docker image: setup docker automated build on Apache DockerHub account

caiok opened a new issue #292: Docker image: setup docker automated build on Apache DockerHub account URL: https://github.com/apache/bookkeeper/issues/292 This issue is just for tracking progresses on this matter This is

[GitHub] caiok commented on issue #288: Docker image: provide a template related way to handle different version

caiok commented on issue #288: Docker image: provide a template related way to handle different version URL: https://github.com/apache/bookkeeper/issues/288#issuecomment-317750524 Simple placeholders substitution is not enough for having a common template for centos and alpine. We need

Re: BookKeeper 4.5.0 performance regression ?

2017-07-25 15:36 GMT+02:00 Enrico Olivelli : > > > 2017-07-25 13:58 GMT+02:00 Enrico Olivelli : > >> I noticed that the "performance" drop in my bench happens contextually to >> the opening of several "RandomAccessFile", to .dx files. >> >> In my bench I

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

caiok commented on issue #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#issuecomment-317744941 @sijie @fpj @merlimat Any other doubts or requests for this PR? We need it merged before proceeding with the related issues (and

Re: BookKeeper 4.5.0 performance regression ?

2017-07-25 13:58 GMT+02:00 Enrico Olivelli : > I noticed that the "performance" drop in my bench happens contextually to > the opening of several "RandomAccessFile", to .dx files. > > In my bench I continue to perform writes and after some time the overall > performance

Re: BookKeeper 4.5.0 performance regression ?

I noticed that the "performance" drop in my bench happens contextually to the opening of several "RandomAccessFile", to .dx files. In my bench I continue to perform writes and after some time the overall performance (latency and throughtput) "degrade" while the bench is running I see that the

Re: EnsemblePlacementPolicy Public API breaking changes

2017-07-24 22:54 GMT+02:00 Sijie Guo : > On Mon, Jul 24, 2017 at 1:29 PM, Enrico Olivelli > wrote: > > > Il lun 24 lug 2017, 19:39 Sijie Guo ha scritto: > > > > > On Mon, Jul 24, 2017 at 5:22 AM, Enrico Olivelli

[GitHub] eolivelli opened a new pull request #291: Issue-290 Add checkstyle:checkstyle to bk-merge script

eolivelli opened a new pull request #291: Issue-290 Add checkstyle:checkstyle to bk-merge script URL: https://github.com/apache/bookkeeper/pull/291 Add checkstyle:checkstyle to the merge script. This will be useful in order to check the patch before every commit on the repo

[GitHub] eolivelli opened a new issue #290: Add checkstyle:checkstyle to bk-merge script

eolivelli opened a new issue #290: Add checkstyle:checkstyle to bk-merge script URL: https://github.com/apache/bookkeeper/issues/290 **FEATURE REQUEST** 1. Please describe the feature you are requesting. I want an utility to run "checkstyle" before merging patches 2.

[GitHub] eolivelli closed pull request #285: Issue 247: Missing JavaDoc for classes

eolivelli closed pull request #285: Issue 247: Missing JavaDoc for classes URL: https://github.com/apache/bookkeeper/pull/285 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] eolivelli closed issue #247: Missing JavaDoc for classes

eolivelli closed issue #247: Missing JavaDoc for classes URL: https://github.com/apache/bookkeeper/issues/247 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] eolivelli closed pull request #286: BOOKKEEPER-1044: Entrylogger is not readding rolled logs back to the logChannelsToFlush list when exception happens while trying to flush rolled logs

eolivelli closed pull request #286: BOOKKEEPER-1044: Entrylogger is not readding rolled logs back to the logChannelsToFlush list when exception happens while trying to flush rolled logs URL: https://github.com/apache/bookkeeper/pull/286

[GitHub] eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs

eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs URL: https://github.com/apache/bookkeeper/pull/276#discussion_r129248338 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.java

[GitHub] eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs

eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs URL: https://github.com/apache/bookkeeper/pull/276#discussion_r129248305 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.java

[GitHub] eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs

eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs URL: https://github.com/apache/bookkeeper/pull/276#discussion_r129248134 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntry.java ##

[GitHub] eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs

eolivelli commented on a change in pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs URL: https://github.com/apache/bookkeeper/pull/276#discussion_r129248110 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntry.java ##

[GitHub] jiazhai opened a new issue #289: Docker image: make docker image build based on master latest code

jiazhai opened a new issue #289: Docker image: make docker image build based on master latest code URL: https://github.com/apache/bookkeeper/issues/289 In the first docker image PR (#197), there is a suggestion to make docker image build based on master latest code

Re: [DISCUSS] Slack Channel for BookKeeper

Awesome +1 Il mar 25 lug 2017, 05:39 Dustin Castor ha scritto: > Agreed! Or an IRC. > > On Monday, July 24, 2017, 6:50:22 PM PDT, Jia Zhai > wrote: > >  It is great to have a slack channel. It make things more effective and > smooth. > > On

[jira] [Commented] (BOOKKEEPER-1016) Create documentation for JAAS and Kerberos configuration

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099581#comment-16099581 ] Enrico Olivelli commented on BOOKKEEPER-1016: - I am sorry. This week I am not sure to

[GitHub] eolivelli commented on issue #183: BOOKKEEPER-588 SSL Support for Bookkeeper

eolivelli commented on issue #183: BOOKKEEPER-588 SSL Support for Bookkeeper URL: https://github.com/apache/bookkeeper/pull/183#issuecomment-317644019 From my point of view it can be deferred to 4.6. I needed zk acls and sasl and all of these features are in