Jenkins build is back to stable : distributedlog-nightly-build #504

See

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

sijie commented on issue #552: Update BookKeeper Tutorial to the new API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-348669531 @ivankelly if they are blocked by the pending BPs, let's move it to 4.7. since the API is unstable in 4.6, it is okay to leave tutorial as it

[GitHub] sijie commented on issue #780: Update Dockerfile in 4.5.1 to use org.apache.bookkeeper.stats.NullStatsProvider

sijie commented on issue #780: Update Dockerfile in 4.5.1 to use org.apache.bookkeeper.stats.NullStatsProvider URL: https://github.com/apache/bookkeeper/issues/780#issuecomment-348669107 #781 is merged. This is an automated

[GitHub] sijie closed issue #780: Update Dockerfile in 4.5.1 to use org.apache.bookkeeper.stats.NullStatsProvider

sijie closed issue #780: Update Dockerfile in 4.5.1 to use org.apache.bookkeeper.stats.NullStatsProvider URL: https://github.com/apache/bookkeeper/issues/780 This is an automated message from the Apache Git Service. To

[GitHub] sijie closed pull request #781: Issue 780: change BK_statsProviderClass in to NullStatsProvider in Docker file

sijie closed pull request #781: Issue 780: change BK_statsProviderClass in to NullStatsProvider in Docker file URL: https://github.com/apache/bookkeeper/pull/781 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] sijie closed pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

sijie closed pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] sijie commented on issue #795: BP-22: Separate closing ledgers from opening ledgers

sijie commented on issue #795: BP-22: Separate closing ledgers from opening ledgers URL: https://github.com/apache/bookkeeper/pull/795#issuecomment-348668404 @ivankelly - There are too many back and forth here. let me try to summarize my opinion here. so it is clear rather than spreading

[GitHub] acoburn opened a new pull request #800: ISSUE 230: Enable checkstyle for more packages

acoburn opened a new pull request #800: ISSUE 230: Enable checkstyle for more packages URL: https://github.com/apache/bookkeeper/pull/800 Part of #230, this enables the checkstyle plugin for the replication and server.http.service packages. Descriptions of the changes in this PR:

[GitHub] ivankelly commented on a change in pull request #798: Removed all per-entry allocations in Journal operations

ivankelly commented on a change in pull request #798: Removed all per-entry allocations in Journal operations URL: https://github.com/apache/bookkeeper/pull/798#discussion_r154454223 ## File path:

[GitHub] acoburn opened a new pull request #799: ISSUE 230: Add Checkstyle to some packages

acoburn opened a new pull request #799: ISSUE 230: Add Checkstyle to some packages URL: https://github.com/apache/bookkeeper/pull/799 This is part of #230 and adds checkstyle verification to the following packages in bookkeeper-server: - metastore - net - sasl

[GitHub] ivankelly commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers

ivankelly commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers URL: https://github.com/apache/bookkeeper/pull/795#discussion_r154436300 ## File path: site/bps/BP-22-separate-closing-ledgers-from-opening-ledgers.md ## @@ -0,0

[GitHub] sijie commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers

sijie commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers URL: https://github.com/apache/bookkeeper/pull/795#discussion_r154413155 ## File path: site/bps/BP-22-separate-closing-ledgers-from-opening-ledgers.md ## @@ -0,0 +1,97

[GitHub] sijie commented on a change in pull request #742: Issue-744 BP-18 introduce write flags

sijie commented on a change in pull request #742: Issue-744 BP-18 introduce write flags URL: https://github.com/apache/bookkeeper/pull/742#discussion_r154391492 ## File path: site/docs/latest/api/ledger-api.md ## @@ -471,3 +471,304 @@ mvn exec:java

[GitHub] sijie commented on a change in pull request #798: Removed all per-entry allocations in Journal operations

sijie commented on a change in pull request #798: Removed all per-entry allocations in Journal operations URL: https://github.com/apache/bookkeeper/pull/798#discussion_r154389940 ## File path:

[GitHub] sijie commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers

sijie commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers URL: https://github.com/apache/bookkeeper/pull/795#discussion_r154388705 ## File path: site/bps/BP-22-separate-closing-ledgers-from-opening-ledgers.md ## @@ -0,0 +1,97

[GitHub] eolivelli closed pull request #792: ISSUE #230: Enable Checkstyle on the proto package

eolivelli closed pull request #792: ISSUE #230: Enable Checkstyle on the proto package URL: https://github.com/apache/bookkeeper/pull/792 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] eolivelli closed pull request #797: ISSUE 230: Enable Checkstyle on the meta package

eolivelli closed pull request #797: ISSUE 230: Enable Checkstyle on the meta package URL: https://github.com/apache/bookkeeper/pull/797 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[GitHub] eolivelli closed pull request #796: ISSUE 230: Enable Checkstyle on the conf package

eolivelli closed pull request #796: ISSUE 230: Enable Checkstyle on the conf package URL: https://github.com/apache/bookkeeper/pull/796 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[GitHub] eolivelli commented on issue #796: ISSUE 230: Enable Checkstyle on the conf package

eolivelli commented on issue #796: ISSUE 230: Enable Checkstyle on the conf package URL: https://github.com/apache/bookkeeper/pull/796#issuecomment-348532600 merging now This is an automated message from the Apache Git

[GitHub] eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags

eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags URL: https://github.com/apache/bookkeeper/pull/742#discussion_r154375957 ## File path: site/docs/latest/api/ledger-api.md ## @@ -471,3 +471,304 @@ mvn exec:java

[GitHub] eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags

eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags URL: https://github.com/apache/bookkeeper/pull/742#discussion_r154375634 ## File path: site/docs/latest/api/ledger-api.md ## @@ -471,3 +471,304 @@ mvn exec:java

[GitHub] eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags

eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags URL: https://github.com/apache/bookkeeper/pull/742#discussion_r154369538 ## File path: site/docs/latest/api/ledger-api.md ## @@ -36,12 +36,12 @@ If you're using

[GitHub] eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags

eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags URL: https://github.com/apache/bookkeeper/pull/742#discussion_r154369521 ## File path: site/docs/latest/api/ledger-api.md ## @@ -20,7 +20,7 @@ If you're using

[GitHub] eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags

eolivelli commented on a change in pull request #742: Issue-744 BP-18 introduce write flags URL: https://github.com/apache/bookkeeper/pull/742#discussion_r154369476 ## File path: site/.ruby-version ## @@ -0,0 +1 @@ +2.4.1 Review comment: it is my local ruby "rbenv",

[GitHub] ivankelly commented on issue #798: Removed all per-entry allocations in Journal operations

ivankelly commented on issue #798: Removed all per-entry allocations in Journal operations URL: https://github.com/apache/bookkeeper/pull/798#issuecomment-348520101 retest This is an automated message from the Apache Git

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

ivankelly commented on issue #552: Update BookKeeper Tutorial to the new API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-348511571 Blocked pending the ongoing API discussions. On Fri, Dec 1, 2017 at 3:26 PM, Jia Zhai wrote:

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

jiazhai commented on issue #552: Update BookKeeper Tutorial to the new API URL: https://github.com/apache/bookkeeper/issues/552#issuecomment-348507838 @ivankelly , what is the status of this issue? :) This is an automated

[GitHub] jiazhai closed issue #673: Document New API introduced in BP-15

jiazhai closed issue #673: Document New API introduced in BP-15 URL: https://github.com/apache/bookkeeper/issues/673 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] jiazhai commented on issue #673: Document New API introduced in BP-15

jiazhai commented on issue #673: Document New API introduced in BP-15 URL: https://github.com/apache/bookkeeper/issues/673#issuecomment-348506803 this is a dup of #553, would like to close this. This is an automated message

[GitHub] jiazhai closed issue #787: Client should be able to get ledger metadata from Handles.

jiazhai closed issue #787: Client should be able to get ledger metadata from Handles. URL: https://github.com/apache/bookkeeper/issues/787 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] jiazhai closed pull request #788: Issue 787: Client should be able to get ledger metadata from Handles.

jiazhai closed pull request #788: Issue 787: Client should be able to get ledger metadata from Handles. URL: https://github.com/apache/bookkeeper/pull/788 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

Jenkins build is back to normal : bookkeeper-master #1961

See

[GitHub] ivankelly closed pull request #774: Open should run recovery by default

ivankelly closed pull request #774: Open should run recovery by default URL: https://github.com/apache/bookkeeper/pull/774 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] ivankelly commented on issue #774: Open should run recovery by default

ivankelly commented on issue #774: Open should run recovery by default URL: https://github.com/apache/bookkeeper/pull/774#issuecomment-348445153 Closing this PR. Related discussion continues on #795 This is an automated

[GitHub] ivankelly commented on a change in pull request #798: Removed all per-entry allocations in Journal operations

ivankelly commented on a change in pull request #798: Removed all per-entry allocations in Journal operations URL: https://github.com/apache/bookkeeper/pull/798#discussion_r154297620 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Journal.java

[GitHub] ivankelly commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers

ivankelly commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers URL: https://github.com/apache/bookkeeper/pull/795#discussion_r154297231 ## File path: site/bps/BP-22-separate-closing-ledgers-from-opening-ledgers.md ## @@ -0,0

[GitHub] ivankelly commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers

ivankelly commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers URL: https://github.com/apache/bookkeeper/pull/795#discussion_r154297057 ## File path: site/bps/BP-22-separate-closing-ledgers-from-opening-ledgers.md ## @@ -0,0

[GitHub] ivankelly commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers

ivankelly commented on a change in pull request #795: BP-22: Separate closing ledgers from opening ledgers URL: https://github.com/apache/bookkeeper/pull/795#discussion_r154295514 ## File path: site/bps/BP-22-separate-closing-ledgers-from-opening-ledgers.md ## @@ -0,0

Jenkins build is back to normal : distributedlog-release-nightly-snapshot #95

See

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

See