[GitHub] sijie commented on a change in pull request #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

sijie commented on a change in pull request #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301#discussion_r129767278 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/client/GenericE

[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_r129767410 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/http/ConfigurationService.java ##

[GitHub] eolivelli commented on a change in pull request #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

eolivelli commented on a change in pull request #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301#discussion_r129768396 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/client/Gene

[GitHub] sijie commented on a change in pull request #297: Issue 296: Bookie supports ephemeral port

sijie commented on a change in pull request #297: Issue 296: Bookie supports ephemeral port URL: https://github.com/apache/bookkeeper/pull/297#discussion_r129768404 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieNettyServer.java ## @@ -219

[GitHub] eolivelli commented on a change in pull request #297: Issue 296: Bookie supports ephemeral port

eolivelli commented on a change in pull request #297: Issue 296: Bookie supports ephemeral port URL: https://github.com/apache/bookkeeper/pull/297#discussion_r129768718 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieNettyServer.java ## @@

[GitHub] sijie commented on a change in pull request #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

sijie commented on a change in pull request #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301#discussion_r129770165 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/client/GenericE

[GitHub] eolivelli closed pull request #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

eolivelli closed pull request #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301 This is an automated message from the Apache Git Service. To

[GitHub] eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

eolivelli commented on issue #301: Issue-299 EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/pull/301#issuecomment-318281496 Closing as won't fix This is an automat

[GitHub] sijie commented on issue #309: Issue-308 LocalBookKeeper.java rethrow Exception in case of failed boot

sijie commented on issue #309: Issue-308 LocalBookKeeper.java rethrow Exception in case of failed boot URL: https://github.com/apache/bookkeeper/pull/309#issuecomment-318281513 Can we keep the log message and rethrow the exception? -

[GitHub] eolivelli closed issue #299: EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

eolivelli closed issue #299: EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/issues/299 This is an automated message from the Apache Git Service. To respond to the

[jira] [Assigned] (BOOKKEEPER-935) Publish sources and javadocs to Maven Central

[ https://issues.apache.org/jira/browse/BOOKKEEPER-935?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo reassigned BOOKKEEPER-935: Assignee: Sijie Guo > Publish sources and javadocs to Maven Central > -

[GitHub] eolivelli commented on issue #299: EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients

eolivelli commented on issue #299: EnsemblePlacementPolicy in 4.5 is not compatible with 4.4 clients URL: https://github.com/apache/bookkeeper/issues/299#issuecomment-318281879 Closing as won't fix. See the discussion on #301 In this case it is really hard to preserve compatibility,

[jira] [Commented] (BOOKKEEPER-935) Publish sources and javadocs to Maven Central

[ https://issues.apache.org/jira/browse/BOOKKEEPER-935?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16102842#comment-16102842 ] Sijie Guo commented on BOOKKEEPER-935: -- [~eolivelli] I will take care of it. > P

[GitHub] eolivelli commented on issue #309: Issue-308 LocalBookKeeper.java rethrow Exception in case of failed boot

eolivelli commented on issue #309: Issue-308 LocalBookKeeper.java rethrow Exception in case of failed boot URL: https://github.com/apache/bookkeeper/pull/309#issuecomment-318282578 @sijie done This is an automated message f

Re: ZooKeeper-less - single bookie setup

it might be better to call it 'local model' or 'standalone mode', rather than 'zookeeper-less'. On Wed, Jul 26, 2017 at 8:26 PM, Enrico Olivelli wrote: > Hi, > I would like to share this idea with you before starting a real design > effort for this new configuration. > Sometimes I have to run

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

I am sorry, I cannot attend. My notes: - switch from 4.4 to 4.5, look for differences, find performances regressions/improvements, check for API compatibility (see various issues/emails from me) - release plan and estimate for 4.5, I hope we could have a release within the start of September (vaca

Re: EnsemblePlacementPolicy Public API breaking changes

The end of the story is that discussing with Sijie we decided to close the issue as won't fix. In the future we will define the rules to handle backward compatibility. Changes on the PlacementPolicy are about semantics from 4.4 to 4.5 Enrico Il mer 26 lug 2017, 12:26 Enrico Olivelli ha scritto:

Build failed in Jenkins: bookkeeper-master #1830

See Changes: [eolivelli] ISSUE #295: LocalBookKeeper fails to start up [eolivelli] ISSUE #271: LedgerHandle#readEntries leaks ByteBufs -- [...truncated 33.56 KB...] Runn

Re: Build failed in Jenkins: bookkeeper-master #1830

Sorry the failed test belongs to my latest commit. Something went wrong while addressing review comments I will fix soon Enrico Il gio 27 lug 2017, 15:51 Apache Jenkins Server ha scritto: > See < > https://builds.apache.org/job/bookkeeper-master/1830/display/redirect?page=changes > > > > Changes

[GitHub] jiazhai closed issue #308: LocalBookKeeper.java rethrow Exception in case of failed boot

jiazhai closed issue #308: LocalBookKeeper.java rethrow Exception in case of failed boot URL: https://github.com/apache/bookkeeper/issues/308 This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] jiazhai closed pull request #309: Issue-308 LocalBookKeeper.java rethrow Exception in case of failed boot

jiazhai closed pull request #309: Issue-308 LocalBookKeeper.java rethrow Exception in case of failed boot URL: https://github.com/apache/bookkeeper/pull/309 This is an automated message from the Apache Git Service. To respo

Re: ZooKeeper-less - single bookie setup

We have inMemoryMetaStore.java already to store metadata locally isn't it? there are some tests that make use of that too. So if you don't need ZK for consensus and you can use this inMemoryMetaStore for metadata store. On Thu, Jul 27, 2017 at 12:42 AM, Sijie Guo wrote: > it might be better to c

[GitHub] kishorekasi commented on a change in pull request #197: BOOKKEEPER-974: Add an official bookkeeper docker image

kishorekasi commented on a change in pull request #197: BOOKKEEPER-974: Add an official bookkeeper docker image URL: https://github.com/apache/bookkeeper/pull/197#discussion_r129880524 ## File path: docker/4.4.0-alpine/Dockerfile ## @@ -0,0 +1,61 @@ +FROM java:openjdk-8-jr

Re: ZooKeeper-less - single bookie setup

Il gio 27 lug 2017, 17:27 Venkateswara Rao Jujjuri ha scritto: > We have inMemoryMetaStore.java already to store metadata locally isn't it? > there are some tests that make use of that too. So if you don't need ZK for > consensus > and you can use this inMemoryMetaStore for metadata store. > Met

Re: ZooKeeper-less - single bookie setup

On Thu, Jul 27, 2017 at 8:48 AM, Enrico Olivelli wrote: > Il gio 27 lug 2017, 17:27 Venkateswara Rao Jujjuri ha > scritto: > > > We have inMemoryMetaStore.java already to store metadata locally isn't > it? > > there are some tests that make use of that too. So if you don't need ZK > for > > cons

[GitHub] sijie commented on issue #305: Issue 305: New Release Guide

sijie commented on issue #305: Issue 305: New Release Guide URL: https://github.com/apache/bookkeeper/pull/305#issuecomment-318416095 please hold on merging this until 4.5.0 is done. 1) there are still TBD items in the guide 2) we need the release manager go through these steps whe

[website] branch asf-site

I created a blank branch 'asf-site' for enabling gitpubsub for the new website. I am also going to put a index.hml as a place holder there and ask INFRA team to enable gitpubsub under bookkeeper.apache.org/test. So we can explore the solution for allowing the co-exists of new website and old docume

Re: [website] branch asf-site

Sure, make sure to place it under the "content/" folder which is where gitpubsub is expecting it to be. -- Matteo Merli On Thu, Jul 27, 2017 at 10:27 AM, Sijie Guo wrote: > I created a blank branch 'asf-site' for enabling gitpubsub for the new > website. I am also going to put a index.hml as a

[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_r129908480 ## File path: bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/HttpServer.java ##

Re: [website] branch asf-site

On Thu, Jul 27, 2017 at 10:29 AM, Matteo Merli wrote: > Sure, make sure to place it under the "content/" folder which is where > gitpubsub is expecting it to be. > Yeah. https://github.com/apache/bookkeeper/blob/asf-site/content/index.html > > -- > Matteo Merli > > > On Thu, Jul 27, 2017 at 10

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

Here are the meeting minutes: https://cwiki.apache.org/confluence/display/BOOKKEEPER/2017-07-27+Meeting+Notes - Sijie On Thu, Jul 27, 2017 at 2:31 AM, Enrico Olivelli wrote: > I am sorry, I cannot attend. > > My notes: > - switch from 4.4 to 4.5, look for differences, find performances > regres

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

Please let me know if I missed anything. - Sijie On Thu, Jul 27, 2017 at 10:46 AM, Sijie Guo wrote: > Here are the meeting minutes: https://cwiki.apache.org/confluence/display/ > BOOKKEEPER/2017-07-27+Meeting+Notes > > - Sijie > > On Thu, Jul 27, 2017 at 2:31 AM, Enrico Olivelli > wrote: > >>

[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-318436548 notes about the docker image from today's meeting - follow what other apache projects are doing [zeppelin](https://issues.

Re: [DISCUSS] Slack Channel for BookKeeper

As we discussed in today's community meeting , I will start a vote thread for that. - Sijie On Mon, Jul 24, 2017 at 5:11 PM, Sijie Guo wrote: > Hi all, > > What do you guys all think about having a dedicated slack

[VOTE] Slack Channel for BookKeeper

Start a vote thread for transferring the DL slack channel to BK. The propose is: - transfer the dl slack channel apachedistributedlog.slack.com to apachebookkeeper.slack.com - the owner will be BookKeeper PMC (priv...@bookkeeper.apache.org) One note: - the slack channel is for informal/immediat

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

Great I am happy to see 4.5 moving on Enrico Il gio 27 lug 2017, 19:47 Sijie Guo ha scritto: > Please let me know if I missed anything. > > - Sijie > > On Thu, Jul 27, 2017 at 10:46 AM, Sijie Guo wrote: > > > Here are the meeting minutes: > https://cwiki.apache.org/confluence/display/ > > BOO

Re: [VOTE] Slack Channel for BookKeeper

+1 non binding Enrico Il gio 27 lug 2017, 20:02 Sijie Guo ha scritto: > Start a vote thread for transferring the DL slack channel to BK. > > The propose is: > > - transfer the dl slack channel apachedistributedlog.slack.com to > apachebookkeeper.slack.com > - the owner will be BookKeeper PMC (p

Re: [VOTE] Slack Channel for BookKeeper

+1 -- Matteo Merli On Thu, Jul 27, 2017 at 11:06 AM, Enrico Olivelli wrote: > +1 non binding > > Enrico > > Il gio 27 lug 2017, 20:02 Sijie Guo ha scritto: > > > Start a vote thread for transferring the DL slack channel to BK. > > > > The propose is: > > > > - transfer the dl slack channel ap

[GitHub] sijie commented on a change in pull request #307: Issue 306: Update SCM info for using mvn:release

sijie commented on a change in pull request #307: Issue 306: Update SCM info for using mvn:release URL: https://github.com/apache/bookkeeper/pull/307#discussion_r129923972 ## File path: pom.xml ## @@ -29,8 +29,30 @@ 4.5.0-SNAPSHOT bookkeeper pom - bookkeeper + A

[GitHub] sijie commented on a change in pull request #307: Issue 306: Update SCM info for using mvn:release

sijie commented on a change in pull request #307: Issue 306: Update SCM info for using mvn:release URL: https://github.com/apache/bookkeeper/pull/307#discussion_r129923997 ## File path: pom.xml ## @@ -29,8 +29,30 @@ 4.5.0-SNAPSHOT bookkeeper pom - bookkeeper + A

[GitHub] eolivelli commented on issue #307: Issue 306: Update SCM info for using mvn:release

eolivelli commented on issue #307: Issue 306: Update SCM info for using mvn:release URL: https://github.com/apache/bookkeeper/pull/307#issuecomment-318450455 Very good This is an automated message from the Apache Git Service

Re: [website] branch asf-site

https://bookkeeper.apache.org/test/content/ is up now. Any html content committed to asf-site branch will automatically show up under /test/content now. I am going to play around .htaccess to manage the redirection for the old documentation. - Sijie On Thu, Jul 27, 2017 at 10:41 AM, Sijie Guo wr

Re: [VOTE] Slack Channel for BookKeeper

+1 On Thu, Jul 27, 2017 at 8:10 PM, Matteo Merli wrote: > +1 > > -- > Matteo Merli > > > On Thu, Jul 27, 2017 at 11:06 AM, Enrico Olivelli > wrote: > >> +1 non binding >> >> Enrico >> >> Il gio 27 lug 2017, 20:02 Sijie Guo ha scritto: >> >> > Start a vote thread for transferring the DL slack c

Re: ZooKeeper-less - single bookie setup

Il gio 27 lug 2017, 18:28 Sijie Guo ha scritto: > On Thu, Jul 27, 2017 at 8:48 AM, Enrico Olivelli > wrote: > > > Il gio 27 lug 2017, 17:27 Venkateswara Rao Jujjuri > ha > > scritto: > > > > > We have inMemoryMetaStore.java already to store metadata locally isn't > > it? > > > there are some te

[GitHub] sigee commented on issue #230: Add Checkstyle to the build process

sigee commented on issue #230: Add Checkstyle to the build process URL: https://github.com/apache/bookkeeper/issues/230#issuecomment-318493429 @sijie, I couldn't finish it. I've got a new project at my job, so I'm very busy, but I'd like to finish this issue as soon as possible. --

[jira] [Created] (BOOKKEEPER-1104) BookieInitializationTest.testWithDiskFullAndAbilityToCreateNewIndexFile testcase is unreliable

Samuel Just created BOOKKEEPER-1104: --- Summary: BookieInitializationTest.testWithDiskFullAndAbilityToCreateNewIndexFile testcase is unreliable Key: BOOKKEEPER-1104 URL: https://issues.apache.org/jira/browse/BOOK

[GitHub] athanatos opened a new pull request #310: BOOKKEEPER-1104: Fix testWithDiskFullAndAbilityToCreateNewIndexFile

athanatos opened a new pull request #310: BOOKKEEPER-1104: Fix testWithDiskFullAndAbilityToCreateNewIndexFile URL: https://github.com/apache/bookkeeper/pull/310 The usage threshhold was chosen to be very close to the actual disk usage at test time. This made the test unnecessarily fragi

[jira] [Updated] (BOOKKEEPER-1100) Add Http Endpoint for Bookkeeper

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-1100: -- Fix Version/s: (was: 4.5.0) 4.6.0 > Add Http Endpoint for Bookkeeper

[GitHub] sijie commented on a change in pull request #305: Issue 305: New Release Guide

sijie commented on a change in pull request #305: Issue 305: New Release Guide URL: https://github.com/apache/bookkeeper/pull/305#discussion_r129986641 ## File path: docs/community/committer/release_guide.md ## @@ -0,0 +1,475 @@ +# Apache BookKeeper Release Guide + +* TOC +

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

😀👍 On Fri, Jul 28, 2017 at 2:06 AM, Enrico Olivelli wrote: > Great > I am happy to see 4.5 moving on > > Enrico > > Il gio 27 lug 2017, 19:47 Sijie Guo ha scritto: > > > Please let me know if I missed anything. > > > > - Sijie > > > > On Thu, Jul 27, 2017 at 10:46 AM, Sijie Guo wrote: > > > >

[jira] [Updated] (BOOKKEEPER-808) Shell cmd to audit the ledgers and list bookie id requires updation

[ https://issues.apache.org/jira/browse/BOOKKEEPER-808?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-808: - Issue Type: New Feature (was: Sub-task) Parent: (was: BOOKKEEPER-639) > Shell cmd

[jira] [Updated] (BOOKKEEPER-807) Document updatecookie and updateledgers admin tool sequence

[ https://issues.apache.org/jira/browse/BOOKKEEPER-807?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-807: - Issue Type: Documentation (was: Sub-task) Parent: (was: BOOKKEEPER-639) > Documen

[jira] [Commented] (BOOKKEEPER-807) Document updatecookie and updateledgers admin tool sequence

[ https://issues.apache.org/jira/browse/BOOKKEEPER-807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104121#comment-16104121 ] Sijie Guo commented on BOOKKEEPER-807: -- Move this as a separate 'documentation ta

[jira] [Resolved] (BOOKKEEPER-639) Provide hostname based bookie and ledger metadata to support IP change

[ https://issues.apache.org/jira/browse/BOOKKEEPER-639?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved BOOKKEEPER-639. -- Resolution: Fixed Fix Version/s: (was: 4.5.0) 4.4.0 The core f

[jira] [Updated] (BOOKKEEPER-818) Make bookie recovery work with recovering multiple bookies

[ https://issues.apache.org/jira/browse/BOOKKEEPER-818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-818: - Fix Version/s: (was: 4.5.0) 4.6.0 > Make bookie recovery work with reco

[jira] [Resolved] (BOOKKEEPER-754) Bookie: flush ledger storage after replaying journal

[ https://issues.apache.org/jira/browse/BOOKKEEPER-754?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved BOOKKEEPER-754. -- Resolution: Won't Fix Fix Version/s: (was: 4.5.0) It was merged with other change

[jira] [Commented] (BOOKKEEPER-979) Bookie recovery should skip replicating empty ledgers.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-979?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104132#comment-16104132 ] Sijie Guo commented on BOOKKEEPER-979: -- move replication related changes to 4.6.0

[jira] [Updated] (BOOKKEEPER-977) ReplicationWorker should choose bookies from cluster rather than using local bookie

[ https://issues.apache.org/jira/browse/BOOKKEEPER-977?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-977: - Fix Version/s: (was: 4.5.0) 4.6.0 > ReplicationWorker should choose boo

[jira] [Commented] (BOOKKEEPER-915) Auto replication should honor ensemble placement policy

[ https://issues.apache.org/jira/browse/BOOKKEEPER-915?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104129#comment-16104129 ] Sijie Guo commented on BOOKKEEPER-915: -- move replication related changes to 4.6.0

[jira] [Commented] (BOOKKEEPER-977) ReplicationWorker should choose bookies from cluster rather than using local bookie

[ https://issues.apache.org/jira/browse/BOOKKEEPER-977?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104131#comment-16104131 ] Sijie Guo commented on BOOKKEEPER-977: -- move replication related changes to 4.6.0

[jira] [Updated] (BOOKKEEPER-979) Bookie recovery should skip replicating empty ledgers.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-979?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-979: - Fix Version/s: (was: 4.5.0) 4.6.0 > Bookie recovery should skip replica

[jira] [Commented] (BOOKKEEPER-818) Make bookie recovery work with recovering multiple bookies

[ https://issues.apache.org/jira/browse/BOOKKEEPER-818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104128#comment-16104128 ] Sijie Guo commented on BOOKKEEPER-818: -- move replication related changes to 4.6.0

[jira] [Commented] (BOOKKEEPER-929) In ReadOnlyLedgerHandle, metadata gets updated in the event of writeclose but lastAddConfirmed and length is not getting updated.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-929?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16104133#comment-16104133 ] Sijie Guo commented on BOOKKEEPER-929: -- since it is minor, moves this to 4.6.0 >

[jira] [Updated] (BOOKKEEPER-915) Auto replication should honor ensemble placement policy

[ https://issues.apache.org/jira/browse/BOOKKEEPER-915?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-915: - Fix Version/s: (was: 4.5.0) 4.6.0 > Auto replication should honor ensem

[jira] [Updated] (BOOKKEEPER-929) In ReadOnlyLedgerHandle, metadata gets updated in the event of writeclose but lastAddConfirmed and length is not getting updated.

[ https://issues.apache.org/jira/browse/BOOKKEEPER-929?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-929: - Fix Version/s: (was: 4.5.0) 4.6.0 > In ReadOnlyLedgerHandle, metadata g

[jira] [Resolved] (BOOKKEEPER-1017) Create documentation for ZooKeeper ACLs

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1017?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved BOOKKEEPER-1017. --- Resolution: Fixed This is merged at https://github.com/apache/bookkeeper/pull/185 > Crea

[jira] [Updated] (BOOKKEEPER-753) Bookie should run garbage collection before startup when all directories became full

[ https://issues.apache.org/jira/browse/BOOKKEEPER-753?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-753: - Fix Version/s: (was: 4.5.0) > Bookie should run garbage collection before startup when all

[GitHub] sijie commented on issue #218: NPE at TestLongZkLedgerIdGenerator.testGenerateLedgerId

sijie commented on issue #218: NPE at TestLongZkLedgerIdGenerator.testGenerateLedgerId URL: https://github.com/apache/bookkeeper/issues/218#issuecomment-318520921 it seems to be flaky test. remove it from 4.5.0 and mark it as 'help-needed' -

[GitHub] sijie commented on issue #195: Update JMX documentation

sijie commented on issue #195: Update JMX documentation URL: https://github.com/apache/bookkeeper/issues/195#issuecomment-318521190 This will be replaced with 'Metric Collection' in https://lucperkins.github.io/bookkeeper/docs/admin/metrics/ #293 Close this as "Won't Fix"

Re: [VOTE] Slack Channel for BookKeeper

+1 On Fri, Jul 28, 2017 at 4:02 AM, atracymartin wrote: > +1 > > > Sent from my T-Mobile 4G LTE Device > Original message From: Sijie Guo > Date: 7/27/17 2:02 PM (GMT-05:00) To: dev@distributedlog.incubator. > apache.org, dev@bookkeeper.apache.org Subject: [VOTE] Slack Channe

Re: BP-11: New BookKeeper Website

Thanks all for the comments: Re "Font Size" - does most of the people feel the font size is too large? Re "Installation" - There is a section about 'download'. It might lack of checking checksums. We can improve it later. Re "mobile devices" - I checked on my phone, it seems to be fine. Luc pu

[GitHub] jiazhai closed pull request #307: Issue 306: Update SCM info for using mvn:release

jiazhai closed pull request #307: Issue 306: Update SCM info for using mvn:release URL: https://github.com/apache/bookkeeper/pull/307 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] jiazhai closed issue #306: update scm information in pom files so we can use maven:release

jiazhai closed issue #306: update scm information in pom files so we can use maven:release URL: https://github.com/apache/bookkeeper/issues/306 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] jiazhai closed issue #296: Bookies supports ephemeral port

jiazhai closed issue #296: Bookies supports ephemeral port URL: https://github.com/apache/bookkeeper/issues/296 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and u

[GitHub] jiazhai closed pull request #297: Issue 296: Bookie supports ephemeral port

jiazhai closed pull request #297: Issue 296: Bookie supports ephemeral port URL: https://github.com/apache/bookkeeper/pull/297 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] sijie opened a new pull request #311: BOOKKEEPER-935: Javadoc improvements and publish sources and javadocs to Maven Central

sijie opened a new pull request #311: BOOKKEEPER-935: Javadoc improvements and publish sources and javadocs to Maven Central URL: https://github.com/apache/bookkeeper/pull/311 Descriptions of the changes in this PR: - sources: attached source jar for each module - javadocs: attach

[jira] [Updated] (BOOKKEEPER-1104) BookieInitializationTest.testWithDiskFullAndAbilityToCreateNewIndexFile testcase is unreliable

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1104?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-1104: -- Fix Version/s: 4.5.0 > BookieInitializationTest.testWithDiskFullAndAbilityToCreateNewIndexF

[GitHub] jiazhai closed pull request #310: BOOKKEEPER-1104: Fix testWithDiskFullAndAbilityToCreateNewIndexFile

jiazhai closed pull request #310: BOOKKEEPER-1104: Fix testWithDiskFullAndAbilityToCreateNewIndexFile URL: https://github.com/apache/bookkeeper/pull/310 This is an automated message from the Apache Git Service. To respond t

[jira] [Resolved] (BOOKKEEPER-1104) BookieInitializationTest.testWithDiskFullAndAbilityToCreateNewIndexFile testcase is unreliable

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1104?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jia Zhai resolved BOOKKEEPER-1104. -- Resolution: Fixed Issue resolved by merging pull request 310 [https://github.co

[GitHub] zhaijack opened a new issue #312: Remove scripts under bin/

zhaijack opened a new issue #312: Remove scripts under bin/ URL: https://github.com/apache/bookkeeper/issues/312 *Problem* the scripts under bin/ don't seem to be used anymore. they seem to come from the period that we were still using diff files for reviews *Action* De

[GitHub] zhaijack opened a new pull request #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional

zhaijack opened a new pull request #313: Issue 300: EnsemblePlacementPolicy must not use com.google.common.base.Optional but java.util.Optional URL: https://github.com/apache/bookkeeper/pull/313 Descriptions of the changes in this PR: use java8 optional instead of guava optional for

[GitHub] zhaijack opened a new issue #314: Enable checkstyle on bookeeper-server

zhaijack opened a new issue #314: Enable checkstyle on bookeeper-server URL: https://github.com/apache/bookkeeper/issues/314 *Problem* Currently checkstyle is disabled for #230 . However there are already two packages whose checkstyle problems are already fixed. We should enable che