[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-317633621 One of the original goals of persisting bookie status was to make it possible in the future to set the bookie to read only through http endpoint

Re: [DISCUSS] Slack Channel for BookKeeper

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 Tue, Jul 25, 2017 at 8:11 AM, Sijie Guo wrote: > Hi all, > > What do you guys all think

[GitHub] jiazhai opened a new issue #288: Docker image: provide a template related way to handle different version

jiazhai opened a new issue #288: Docker image: provide a template related way to handle different version URL: https://github.com/apache/bookkeeper/issues/288 In the first docker image PR (#197), there is a suggestion to use template and generate Docker images based on different image

Re: [DISCUSS] Slack Channel for BookKeeper

 It is great to have a slack channel. It make things more effective and smooth. On Tue, Jul 25, 2017 at 8:11 AM, Sijie Guo wrote: > Hi all, > > What do you guys all think about having a dedicated slack channel for > informal discussion for the community? There are a handful

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

yzang commented on issue #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278#issuecomment-317596375 Sure, I'll address the styles issues This is an automated

[GitHub] sijie opened a new issue #287: Introduce BookieClusterManager and Improve BookieListener

sijie opened a new issue #287: Introduce BookieClusterManager and Improve BookieListener URL: https://github.com/apache/bookkeeper/issues/287 **FEATURE REQUEST** 1. Please describe the feature you are requesting. - introduce BookieClusterManager class for managing the

[GitHub] sijie commented on issue #287: Introduce BookieClusterManager and Improve BookieListener

sijie commented on issue #287: Introduce BookieClusterManager and Improve BookieListener URL: https://github.com/apache/bookkeeper/issues/287#issuecomment-317604117 tag @yzang as the contributor. This is an automated

[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=16099336#comment-16099336 ] Sijie Guo commented on BOOKKEEPER-807: -- Defer this to new documentation. > Document updatecookie

[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=16099335#comment-16099335 ] Sijie Guo commented on BOOKKEEPER-1016: --- [~eolivelli] are you working on this? If no, I'd happy

[jira] [Assigned] (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 reassigned BOOKKEEPER-807: Assignee: Sijie Guo > Document updatecookie and updateledgers admin tool sequence >

[jira] [Assigned] (BOOKKEEPER-1044) Entrylogger is not readding rolled logs back to the logChannelsToFlush list when exception happens while trying to flush rolled logs

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1044?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo reassigned BOOKKEEPER-1044: - Assignee: Sijie Guo (was: Charan Reddy Guttapalem) I attached a simple fix at

[GitHub] sijie opened a new 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

sijie opened a new 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 Descriptions of the changes in this PR:

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

sijie commented on issue #183: BOOKKEEPER-588 SSL Support for Bookkeeper URL: https://github.com/apache/bookkeeper/pull/183#issuecomment-317592597 @kishorekasi @eolivelli what is the plan for this? Are we including this in 4.5.0?

[jira] [Commented] (BOOKKEEPER-1057) Do not log error message after read failure in PendingReadOp

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099316#comment-16099316 ] Sijie Guo commented on BOOKKEEPER-1057: --- discussed with Matteo and moved this from 4.5.0 to

[jira] [Updated] (BOOKKEEPER-1057) Do not log error message after read failure in PendingReadOp

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1057?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-1057: -- Fix Version/s: (was: 4.5.0) 4.6.0 > Do not log error message after

[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=16099314#comment-16099314 ] Sijie Guo commented on BOOKKEEPER-929: -- [~jujjuri] shall we include this in 4.5.0? or shall we

[jira] [Reopened] (BOOKKEEPER-992) ReadLog Command Enhancement

[ https://issues.apache.org/jira/browse/BOOKKEEPER-992?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo reopened BOOKKEEPER-992: -- > ReadLog Command Enhancement > --- > > Key:

[jira] [Resolved] (BOOKKEEPER-992) ReadLog Command Enhancement

[ https://issues.apache.org/jira/browse/BOOKKEEPER-992?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved BOOKKEEPER-992. -- Resolution: Fixed > ReadLog Command Enhancement > --- > >

[jira] [Commented] (BOOKKEEPER-1062) AuditorLedgerCheckerTest intermittent failures

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099312#comment-16099312 ] Sijie Guo commented on BOOKKEEPER-1062: --- Moved this out of 4.5.0 > AuditorLedgerCheckerTest

[jira] [Updated] (BOOKKEEPER-1062) AuditorLedgerCheckerTest intermittent failures

[ https://issues.apache.org/jira/browse/BOOKKEEPER-1062?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-1062: -- Fix Version/s: (was: 4.5.0) > AuditorLedgerCheckerTest intermittent failures >

[jira] [Commented] (BOOKKEEPER-817) Bookie: LRU FileInfo Eviction Policy

[ https://issues.apache.org/jira/browse/BOOKKEEPER-817?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16099310#comment-16099310 ] Sijie Guo commented on BOOKKEEPER-817: -- Move this out of 4.5.0. [~yzang] will help port changes

[jira] [Updated] (BOOKKEEPER-817) Bookie: LRU FileInfo Eviction Policy

[ https://issues.apache.org/jira/browse/BOOKKEEPER-817?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo updated BOOKKEEPER-817: - Fix Version/s: (was: 4.5.0) > Bookie: LRU FileInfo Eviction Policy >

[jira] [Resolved] (BOOKKEEPER-771) Bookie failed to restart if there is partially written index page on a ledger

[ https://issues.apache.org/jira/browse/BOOKKEEPER-771?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sijie Guo resolved BOOKKEEPER-771. -- Resolution: Won't Fix Fix Version/s: (was: 4.5.0) Cancel this one since this has

[DISCUSS] Slack Channel for BookKeeper

Hi all, What do you guys all think about having a dedicated slack channel for informal discussion for the community? There are a handful of Apache projects are doing that already, there are also ways to have a bot that sends daily digest of the conversation to the mailing lists (to keep the

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

sijie commented on issue #230: Add Checkstyle to the build process URL: https://github.com/apache/bookkeeper/issues/230#issuecomment-317587408 ping @sigee This is an automated message from the Apache Git Service. To respond

[GitHub] sijie opened a new issue #284: Needs to handle AuthFailed, SaslAuthenticated events

sijie opened a new issue #284: Needs to handle AuthFailed, SaslAuthenticated events URL: https://github.com/apache/bookkeeper/issues/284 **FEATURE REQUEST** 1. Please describe the feature you are requesting. Address TODO item in *ZooKeeperWatcherBase*: // TODO:

[GitHub] sijie opened a new issue #283: V2 ReadEntryProcessor should handle asynchronous fence like what V3 does

sijie opened a new issue #283: V2 ReadEntryProcessor should handle asynchronous fence like what V3 does URL: https://github.com/apache/bookkeeper/issues/283 **FEATURE REQUEST** 1. Please describe the feature you are requesting. Address TODO item in *ReadEntryProcessor*:

[GitHub] sijie opened a new issue #282: Record metadata store version somewhere

sijie opened a new issue #282: Record metadata store version somewhere URL: https://github.com/apache/bookkeeper/issues/282 **FEATURE REQUEST** 1. Please describe the feature you are requesting. Address the TODO item in MSLedgerManagerFactory // load metadata

[GitHub] sijie opened a new issue #281: make ledger metadata immutable

sijie opened a new issue #281: make ledger metadata immutable URL: https://github.com/apache/bookkeeper/issues/281 **FEATURE REQUEST** 1. Please describe the feature you are requesting. Address the TODO item in *LedgerHandle* if (isClosed()) {

[GitHub] sijie opened a new issue #280: Turn bookie into readonly when SortedLedgerStorage fails to flush data

sijie opened a new issue #280: Turn bookie into readonly when SortedLedgerStorage fails to flush data URL: https://github.com/apache/bookkeeper/issues/280 **FEATURE REQUEST** 1. Please describe the feature you are requesting. Address the TODO item in `SortedLedgerStorage`

[GitHub] sijie opened a new issue #279: Provide LSN in Journal for better checkpointing

sijie opened a new issue #279: Provide LSN in Journal for better checkpointing URL: https://github.com/apache/bookkeeper/issues/279 **FEATURE REQUEST** 1. Please describe the feature you are requesting. Address the TODO item in **InterleavedLedgerStorage**

Re: EnsemblePlacementPolicy Public API breaking changes

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 > > wrote: > > > > > Hi, > > > we are going to release 4.5 soon, in

Re: BookKeeper 4.5.0 performance regression ?

Il lun 24 lug 2017, 19:54 Venkateswara Rao Jujjuri ha scritto: > On Mon, Jul 24, 2017 at 3:06 AM, Enrico Olivelli > wrote: > > > 2017-07-21 20:37 GMT+02:00 Enrico Olivelli : > > > > > > > > > > > Il ven 21 lug 2017, 20:32 Sijie Guo

Re: EnsemblePlacementPolicy Public API breaking changes

Il lun 24 lug 2017, 19:39 Sijie Guo ha scritto: > On Mon, Jul 24, 2017 at 5:22 AM, Enrico Olivelli > wrote: > > > Hi, > > we are going to release 4.5 soon, in this release we changed > > EnsemblePlacementPolicy interface. > > > > It you are using a

[GitHub] sijie commented on issue #247: Missing JavaDoc for classes

sijie commented on issue #247: Missing JavaDoc for classes URL: https://github.com/apache/bookkeeper/issues/247#issuecomment-317527918 picking this up for unblocking 4.5.0 release. This is an automated message from the

[GitHub] yzang opened a new pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint

yzang opened a new pull request #278: BOOKKEEPER-1100: Add module for Bookkeeper Http Endpoint URL: https://github.com/apache/bookkeeper/pull/278 Add Bookkeeper Http Server as a new module [bookkeeper-http] 1. Add children module [http-server], which provide generic api for setting

[GitHub] jvrao opened a new issue #277: IO Hints

jvrao opened a new issue #277: IO Hints URL: https://github.com/apache/bookkeeper/issues/277 Is this a question, feature request, or bug report? **FEATURE REQUEST** 1. Please describe the feature you are requesting. Its well known practice to send hints down to inform

Re: BookKeeper 4.5.0 performance regression ?

On Mon, Jul 24, 2017 at 3:06 AM, Enrico Olivelli wrote: > 2017-07-21 20:37 GMT+02:00 Enrico Olivelli : > > > > > > > Il ven 21 lug 2017, 20:32 Sijie Guo ha scritto: > > > >> As the discussion in a separate thread, it might be making

[GitHub] sijie commented on issue #274: Write benchmark fails with `BkNotEnoughBookiesException` even when enough bookies are available

sijie commented on issue #274: Write benchmark fails with `BkNotEnoughBookiesException` even when enough bookies are available URL: https://github.com/apache/bookkeeper/issues/274#issuecomment-317502883 @arvindkandhare are you going to provide the fix for this?

[GitHub] sijie commented on a change in pull request #206: Issue-196 Create Jenkins configuration using OpenStack Jenkins Job Builder

sijie commented on a change in pull request #206: Issue-196 Create Jenkins configuration using OpenStack Jenkins Job Builder URL: https://github.com/apache/bookkeeper/pull/206#discussion_r129107315 ## File path: jenkins/README.md ## @@ -0,0 +1,65 @@ + + +Instructions for

[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_r129106589 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.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_r129105945 ## 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_r129106352 ## 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_r129106275 ## 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_r129106853 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ClientConfiguration.java

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

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

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

sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r129104958 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java ## @@ -149,6 +150,8

Re: EnsemblePlacementPolicy Public API breaking changes

On Mon, Jul 24, 2017 at 5:22 AM, Enrico Olivelli wrote: > Hi, > we are going to release 4.5 soon, in this release we changed > EnsemblePlacementPolicy interface. > > It you are using a "custom" EnsemblePlacementPolicy in 4.4 and you just > switch the bookeeeper-server JAR on

[GitHub] eolivelli opened a new pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs

eolivelli opened a new pull request #276: Issue-271 LedgerHandle#readEntries leaks ByteBufs URL: https://github.com/apache/bookkeeper/pull/276 Add a setNettyUsePooledBuffers() client configuration option to let the user decide to use Pooled vs Unpooled ByteBufs. Using v2 wire protocol

[GitHub] eolivelli commented on issue #271: LedgerHandle#readEntries leaks ByteBufs if getEntry not called or getEntryInputStream() not called and closed

eolivelli commented on issue #271: LedgerHandle#readEntries leaks ByteBufs if getEntry not called or getEntryInputStream() not called and closed URL: https://github.com/apache/bookkeeper/issues/271#issuecomment-317443590 After discussion on the mailing list we decided to: - create a

[GitHub] eolivelli commented on issue #272: Provide a minimal test case using v2wireprotocol option

eolivelli commented on issue #272: Provide a minimal test case using v2wireprotocol option URL: https://github.com/apache/bookkeeper/issues/272#issuecomment-317442878 closed with 626ce687e2342b8a75005cf018540fda76ee15d8

[GitHub] eolivelli closed issue #272: Provide a minimal test case using v2wireprotocol option

eolivelli closed issue #272: Provide a minimal test case using v2wireprotocol option URL: https://github.com/apache/bookkeeper/issues/272 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] eolivelli closed issue #243: asyncAddEntry fails with NPE with LedgerHandler created with createLedgerAdv

eolivelli closed issue #243: asyncAddEntry fails with NPE with LedgerHandler created with createLedgerAdv URL: https://github.com/apache/bookkeeper/issues/243 This is an automated message from the Apache Git Service. To

[GitHub] eolivelli closed pull request #244: Issue#243 - asyncAddEntry fails with NPE with LedgerHandlerAdv

eolivelli closed pull request #244: Issue#243 - asyncAddEntry fails with NPE with LedgerHandlerAdv URL: https://github.com/apache/bookkeeper/pull/244 This is an automated message from the Apache Git Service. To respond to

[GitHub] eolivelli commented on issue #275: BOOKKEEPER-1102: Fix BookKeeeperDiskSpaceWeightedLedgerPlacementTest

eolivelli commented on issue #275: BOOKKEEPER-1102: Fix BookKeeeperDiskSpaceWeightedLedgerPlacementTest URL: https://github.com/apache/bookkeeper/pull/275#issuecomment-317408320 @athanatos is this your current code at Salesforce or is it just a proposal ? as far as I know you are the only

EnsemblePlacementPolicy Public API breaking changes

Hi, we are going to release 4.5 soon, in this release we changed EnsemblePlacementPolicy interface. It you are using a "custom" EnsemblePlacementPolicy in 4.4 and you just switch the bookeeeper-server JAR on the classpath your application won't run. I already got that problem and I needed to

[GitHub] eolivelli closed pull request #273: Provide a minimal test case using v2wireprotocol option

eolivelli closed pull request #273: Provide a minimal test case using v2wireprotocol option URL: https://github.com/apache/bookkeeper/pull/273 This is an automated message from the Apache Git Service. To respond to the

Re: BookKeeper 4.5.0 performance regression ?

2017-07-21 20:37 GMT+02:00 Enrico Olivelli : > > > Il ven 21 lug 2017, 20:32 Sijie Guo ha scritto: > >> As the discussion in a separate thread, it might be making sense to check >> what is the difference between using pooled allocator and unpooled >>

[GitHub] yzang commented on issue #210: BOOKKEEPER-1100: Add Http Server for Bookkeeper

yzang commented on issue #210: BOOKKEEPER-1100: Add Http Server for Bookkeeper URL: https://github.com/apache/bookkeeper/pull/210#issuecomment-317365523 Closed this and will open another one which move the HTTP server into a separate module

[GitHub] yzang closed pull request #210: BOOKKEEPER-1100: Add Http Server for Bookkeeper

yzang closed pull request #210: BOOKKEEPER-1100: Add Http Server for Bookkeeper URL: https://github.com/apache/bookkeeper/pull/210 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] nicoloboschi commented on a change in pull request #206: Issue-196 Create Jenkins configuration using OpenStack Jenkins Job Builder

nicoloboschi commented on a change in pull request #206: Issue-196 Create Jenkins configuration using OpenStack Jenkins Job Builder URL: https://github.com/apache/bookkeeper/pull/206#discussion_r128967521 ## File path: jenkins/README.md ## @@ -0,0 +1,65 @@ + +