[GitHub] sijie commented on issue #552: Update BookKeeper Tutorial to the new API

2017-11-15 Thread GitBox
sijie commented on issue #552: Update BookKeeper Tutorial to the new API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-344839993 I haven't started yet. I have other 4.6 tasks pending. I will try to see how do they go on. We can sync offline to spread this task between

Jenkins build became unstable: distributedlog-nightly-build #488

2017-11-15 Thread Apache Jenkins Server
See

[GitHub] asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#issuecomment-344837092 FAILURE --none-- This is an

[GitHub] ivankelly commented on issue #552: Update BookKeeper Tutorial to the new API

2017-11-15 Thread GitBox
ivankelly commented on issue #552: Update BookKeeper Tutorial to the new API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-344836991 @sijie Almost picked it up yesterday, but didn't in the end. Feel free to take it.

[GitHub] sijie commented on issue #732: Release 4.5.1: update docker image file after 4.5.1 is released

2017-11-15 Thread GitBox
sijie commented on issue #732: Release 4.5.1: update docker image file after 4.5.1 is released URL: https://github.com/apache/bookkeeper/issues/732#issuecomment-344836384 @eolivelli just a reminder you need to update docker image file in branch-4.5 for release 4.5.1. so dockerhub can

[GitHub] jvrao commented on issue #471: BP-14 Relax Durability

2017-11-15 Thread GitBox
jvrao commented on issue #471: BP-14 Relax Durability URL: https://github.com/apache/bookkeeper/issues/471#issuecomment-344833882 Good idea and practice to keep the BP up to date and documenting API along with the feature. do not send ledger typple

[GitHub] sijie commented on issue #552: Update BookKeeper Tutorial to the the API

2017-11-15 Thread GitBox
sijie commented on issue #552: Update BookKeeper Tutorial to the the API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-344833427 @jvrao sure I will add this one. This is an automated message from the

[GitHub] sijie commented on issue #670: BP-14 Relax Durability - patch 1 Introducing LedgerType

2017-11-15 Thread GitBox
sijie commented on issue #670: BP-14 Relax Durability - patch 1 Introducing LedgerType URL: https://github.com/apache/bookkeeper/pull/670#issuecomment-344833180 As I pointed out in the master ticket of BP-14 #471, we need documentation for this pull request to be able to be merged, or at

[GitHub] jiazhai opened a new issue #733: fix getKeyName in ComponentConfiguration.

2017-11-15 Thread GitBox
jiazhai opened a new issue #733: fix getKeyName in ComponentConfiguration. URL: https://github.com/apache/bookkeeper/issues/733 In getKeyName, there seems missed a "." between "componentPrefix" and "keyName", when a child class passed in a componentPrefix, like "compX", then the

[GitHub] eolivelli commented on issue #471: BP-14 Relax Durability

2017-11-15 Thread GitBox
eolivelli commented on issue #471: BP-14 Relax Durability URL: https://github.com/apache/bookkeeper/issues/471#issuecomment-344832889 Yes, I undestand. Will do This is an automated message from the Apache Git Service. To

[GitHub] sijie commented on issue #471: BP-14 Relax Durability

2017-11-15 Thread GitBox
sijie commented on issue #471: BP-14 Relax Durability URL: https://github.com/apache/bookkeeper/issues/471#issuecomment-344832510 @eolivelli - since this is a big feature, I would like to make sure documentation is made along with this BP. I would like to see a plan for adding the

[GitHub] eolivelli commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151330821 ## File path:

[GitHub] eolivelli commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151330828 ## File path:

[GitHub] jvrao commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
jvrao commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#issuecomment-344829163 LGTM This is an automated message from the

[GitHub] sijie commented on issue #552: Update BookKeeper Tutorial to the the API

2017-11-15 Thread GitBox
sijie commented on issue #552: Update BookKeeper Tutorial to the the API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-344829122 well I think it is sort of required for 4.6. You need to demonstrate the usage of new API for people, even it is experimental. I would like

[GitHub] eolivelli commented on issue #552: Update BookKeeper Tutorial to the the API

2017-11-15 Thread GitBox
eolivelli commented on issue #552: Update BookKeeper Tutorial to the the API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-344825825 Not at the moment. I think this can be for 4.7, there is no hurry, the API is still experimental

[GitHub] zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151327508 ## File path:

[GitHub] zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151327487 ## File path:

[GitHub] zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151327490 ## File path:

[GitHub] zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151327430 ## File path:

[GitHub] zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151327395 ## File path:

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151327009 ## File path:

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151326916 ## File path:

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151326934 ## File path:

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151326870 ## File path:

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151327022 ## File path:

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151326929 ## File path:

[GitHub] asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#issuecomment-344815234 FAILURE --none-- This is an

[GitHub] asfgit commented on issue #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
asfgit commented on issue #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#issuecomment-344810841 FAILURE --none-- This is an automated message

[GitHub] jiazhai opened a new issue #732: Release 4.5.1: update docker image file after 4.5.1 is released

2017-11-15 Thread GitBox
jiazhai opened a new issue #732: Release 4.5.1: update docker image file after 4.5.1 is released URL: https://github.com/apache/bookkeeper/issues/732 update docker image file after 4.5.1 is released This is an automated

[GitHub] zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151311828 ## File path:

[GitHub] zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151311751 ## File path:

[GitHub] zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151311549 ## File path:

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151305313 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntries.java

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151305502 ## File path:

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151305313 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntries.java

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151305208 ## File path:

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151304990 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntries.java

[GitHub] jiazhai opened a new issue #731: Refine LedgerEntry interface

2017-11-15 Thread GitBox
jiazhai opened a new issue #731: Refine LedgerEntry interface URL: https://github.com/apache/bookkeeper/issues/731 In review of #727, @sijie suggested to refine the LedgerEntry interface: The LedgerEntry interface is completely a mess. getEntry() and getEntryBuffer() have

[GitHub] sijie commented on issue #552: Update BookKeeper Tutorial to the the API

2017-11-15 Thread GitBox
sijie commented on issue #552: Update BookKeeper Tutorial to the the API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-344788327 @eolivelli @ivankelly is any of you working on this? This is an automated

[GitHub] asfgit commented on issue #730: Issue 726: Unit Tests failure with BKMetadataVersionException

2017-11-15 Thread GitBox
asfgit commented on issue #730: Issue 726: Unit Tests failure with BKMetadataVersionException URL: https://github.com/apache/bookkeeper/pull/730#issuecomment-344787789 FAILURE --none-- This is an automated message

[GitHub] asfgit commented on issue #730: Issue 726: Unit Tests failure with BKMetadataVersionException

2017-11-15 Thread GitBox
asfgit commented on issue #730: Issue 726: Unit Tests failure with BKMetadataVersionException URL: https://github.com/apache/bookkeeper/pull/730#issuecomment-344771559 FAILURE --none-- This is an automated message

[GitHub] sijie opened a new pull request #730: Issue 726: Unit Tests failure with BKMetadataVersionException

2017-11-15 Thread GitBox
sijie opened a new pull request #730: Issue 726: Unit Tests failure with BKMetadataVersionException URL: https://github.com/apache/bookkeeper/pull/730 Descriptions of the changes in this PR: Problem: The problem is introduced after making storing ctime optional. the

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151261333 ## File path:

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151261724 ## File path:

[GitHub] sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
sijie commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151260556 ## File path:

[GitHub] eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151228183 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntries.java

[GitHub] eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151228170 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntries.java

[GitHub] sijie commented on issue #713: New apis should be experimental until they've been used a bit

2017-11-15 Thread GitBox
sijie commented on issue #713: New apis should be experimental until they've been used a bit URL: https://github.com/apache/bookkeeper/issues/713#issuecomment-344701128 All the api should be marked as `@InterfaceAudience.Public` and `@InterfaceStability.Unstable`

[GitHub] sijie commented on a change in pull request #728: Issue-553 Documentation for new API (WIP)

2017-11-15 Thread GitBox
sijie commented on a change in pull request #728: Issue-553 Documentation for new API (WIP) URL: https://github.com/apache/bookkeeper/pull/728#discussion_r151227038 ## File path: site/docs/4.6.0/api/ledger-api.md ## @@ -264,6 +264,88 @@ Result: 2 # etc ``` +## New API

[GitHub] sijie commented on issue #728: Issue-553 Documentation for new API (WIP)

2017-11-15 Thread GitBox
sijie commented on issue #728: Issue-553 Documentation for new API (WIP) URL: https://github.com/apache/bookkeeper/pull/728#issuecomment-344699622 The API documentation should be the final change after we sort out all the api related issues.

[GitHub] sijie commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
sijie commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151218740 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntries.java

[GitHub] sijie commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
sijie commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151225622 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/SyncCallbackUtils.java

[GitHub] sijie commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
sijie commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151225239 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerEntries.java

[GitHub] sijie commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
sijie commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151218118 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/api/LedgerEntries.java

[GitHub] asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#issuecomment-344626589 FAILURE --none-- This is an

[GitHub] asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#issuecomment-344623160 FAILURE --none-- This is an

[GitHub] asfgit commented on issue #728: Issue-553 Documentation for new API (WIP)

2017-11-15 Thread GitBox
asfgit commented on issue #728: Issue-553 Documentation for new API (WIP) URL: https://github.com/apache/bookkeeper/pull/728#issuecomment-344615820 FAILURE --none-- This is an automated message from the Apache Git

[GitHub] eolivelli commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151145008 ## File path:

[GitHub] eolivelli commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#discussion_r151144498 ## File path:

[GitHub] asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
asfgit commented on issue #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729#issuecomment-344612062 FAILURE --none-- This is an

[GitHub] jiazhai commented on issue #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on issue #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#issuecomment-344606742 @eolivelli, Regarding the release/retain story, Here is the understanding: Before this change, read() return result

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151134947 ## File path:

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151133108 ## File path:

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151132080 ## File path:

[GitHub] jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
jiazhai commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151131418 ## File path:

[GitHub] jiazhai commented on issue #728: Issue-553 Documentation for new API (WIP)

2017-11-15 Thread GitBox
jiazhai commented on issue #728: Issue-553 Documentation for new API (WIP) URL: https://github.com/apache/bookkeeper/pull/728#issuecomment-344598071 Seems #727 is going on, could we merge this after that, and change some of the description according to it?

[GitHub] jiazhai commented on issue #728: Issue-553 Documentation for new API (WIP)

2017-11-15 Thread GitBox
jiazhai commented on issue #728: Issue-553 Documentation for new API (WIP) URL: https://github.com/apache/bookkeeper/pull/728#issuecomment-344598071 Seems #727 is going on, should we merge this after that, and change some of the description according to it?

[GitHub] zhaijack opened a new pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read

2017-11-15 Thread GitBox
zhaijack opened a new pull request #729: Issue 550: add readLastAddConfirmedAndEntry in ReadHandle for long poll read URL: https://github.com/apache/bookkeeper/pull/729 Descriptions of the changes in this PR: 1, add a class LastAddConfirmedAndEntry and metnod

[GitHub] eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151123107 ## File path:

[GitHub] eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151123144 ## File path:

[GitHub] eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151124026 ## File path:

[GitHub] eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
eolivelli commented on a change in pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#discussion_r151123397 ## File path:

Build failed in Jenkins: bookkeeper-master #1942

2017-11-15 Thread Apache Jenkins Server
See Changes: [sijie] Bookies should not queue read request indefinitely [zhaijia] ISSUE #724: fix wrong link for slack signup -- [...truncated 52.92 KB...] Tests run:

[GitHub] eolivelli commented on issue #728: Issue-553 Documentation for new API (WIP)

2017-11-15 Thread GitBox
eolivelli commented on issue #728: Issue-553 Documentation for new API (WIP) URL: https://github.com/apache/bookkeeper/pull/728#issuecomment-344587959 @sijie can you take a first look ? This is an automated message from the

[GitHub] eolivelli opened a new pull request #728: Issue-553 Documentation for new API (WIP)

2017-11-15 Thread GitBox
eolivelli opened a new pull request #728: Issue-553 Documentation for new API (WIP) URL: https://github.com/apache/bookkeeper/pull/728 Add documentation for new API This is an automated message from the Apache Git Service.

[GitHub] asfgit commented on issue #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
asfgit commented on issue #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727#issuecomment-344582207 FAILURE --none-- This is an automated message

[GitHub] jiazhai commented on issue #550: Support Long-poll reads in new API

2017-11-15 Thread GitBox
jiazhai commented on issue #550: Support Long-poll reads in new API URL: https://github.com/apache/bookkeeper/issues/550#issuecomment-344569275 will do it. This is an automated message from the Apache Git Service. To respond

[GitHub] zhaijack opened a new pull request #727: Issue 693: add interface and implementation of LedgerEntries

2017-11-15 Thread GitBox
zhaijack opened a new pull request #727: Issue 693: add interface and implementation of LedgerEntries URL: https://github.com/apache/bookkeeper/pull/727 Descriptions of the changes in this PR: 1, add interface LedgerEntries; 2, add implemtation of LedgerEntries; 3, fix related

[GitHub] asfgit commented on issue #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie

2017-11-15 Thread GitBox
asfgit commented on issue #712: Issue 544: Bootup cookie validation considers an empty journal to signify a new bookie URL: https://github.com/apache/bookkeeper/pull/712#issuecomment-344545727 FAILURE --none-- This

[GitHub] ivankelly opened a new issue #238: Quickstart is out of date with regard to available binaries

2017-11-15 Thread GitBox
ivankelly opened a new issue #238: Quickstart is out of date with regard to available binaries URL: https://github.com/apache/distributedlog/issues/238 The quickstart refers to a "all" binary package, which doesn't exist for 0.5.0, (there's only proxy-server and benchmark). The proxy

[GitHub] jiazhai commented on issue #726: Unit Tests failure with BKMetadataVersionException

2017-11-15 Thread GitBox
jiazhai commented on issue #726: Unit Tests failure with BKMetadataVersionException URL: https://github.com/apache/bookkeeper/issues/726#issuecomment-344528515 Thanks for @sijie's help, this may related changes in #718 This

Jenkins build became unstable: distributedlog-release-nightly-snapshot #80

2017-11-15 Thread Apache Jenkins Server
See

Jenkins build is still unstable: bookkeeper-release-nightly-snapshot #103

2017-11-15 Thread Apache Jenkins Server
See