[GitHub] sijie closed pull request #1311: Issue #1310: Filter out body and masterKey from ProcessorV3s

sijie closed pull request #1311: Issue #1310: Filter out body and masterKey from ProcessorV3s URL: https://github.com/apache/bookkeeper/pull/1311 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

[GitHub] sijie closed issue #1310: filter out confidential data (entry body and masterkey) from ProcessorV3 toString representation

sijie closed issue #1310: filter out confidential data (entry body and masterkey) from ProcessorV3 toString representation URL: https://github.com/apache/bookkeeper/issues/1310 This is an automated message from the Apache Gi

[GitHub] sijie closed pull request #1298: newOpenLedgerOp and ReadHandle implementation for MockBookKeeper

sijie closed pull request #1298: newOpenLedgerOp and ReadHandle implementation for MockBookKeeper URL: https://github.com/apache/bookkeeper/pull/1298 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:

[GitHub] sijie closed issue #1314: Allow ledger recovery writes on readonly bookies

sijie closed issue #1314: Allow ledger recovery writes on readonly bookies URL: https://github.com/apache/bookkeeper/issues/1314 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] sijie closed pull request #1315: Issue 1314: Provide a mechanism to allow high priority writes to readonly bookies

sijie closed pull request #1315: Issue 1314: Provide a mechanism to allow high priority writes to readonly bookies URL: https://github.com/apache/bookkeeper/pull/1315 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sa

[GitHub] eolivelli closed issue #569: Run BookKeeper on Java9

eolivelli closed issue #569: Run BookKeeper on Java9 URL: https://github.com/apache/bookkeeper/issues/569 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the U

[GitHub] eolivelli commented on issue #569: Run BookKeeper on Java9

eolivelli commented on issue #569: Run BookKeeper on Java9 URL: https://github.com/apache/bookkeeper/issues/569#issuecomment-378849738 Now (4.6.2) BookKeeper is running on jdk10 (and jdk9). We can close this issue This is an

[GitHub] eolivelli opened a new pull request #1317: WIP - BP-14 Force Ledger Bookie side implementation

eolivelli opened a new pull request #1317: WIP - BP-14 Force Ledger Bookie side implementation URL: https://github.com/apache/bookkeeper/pull/1317 Implement "force" API on journal and bookie side. In BP-14 the client will have a new force(long ledgerId) API which ensures that every

[GitHub] eolivelli commented on issue #1317: WIP - BP-14 Force Ledger Bookie side implementation

eolivelli commented on issue #1317: WIP - BP-14 Force Ledger Bookie side implementation URL: https://github.com/apache/bookkeeper/pull/1317#issuecomment-378852184 This patch is still WIP, as it lacks important features/fixes: - after Bookie restart the LastPersistedEntryId is read from L

[GitHub] eolivelli commented on a change in pull request #1317: WIP - BP-14 Force Ledger Bookie side implementation

eolivelli commented on a change in pull request #1317: WIP - BP-14 Force Ledger Bookie side implementation URL: https://github.com/apache/bookkeeper/pull/1317#discussion_r179378493 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java ## @

[GitHub] eolivelli commented on a change in pull request #1317: WIP - BP-14 Force Ledger Bookie side implementation

eolivelli commented on a change in pull request #1317: WIP - BP-14 Force Ledger Bookie side implementation URL: https://github.com/apache/bookkeeper/pull/1317#discussion_r179378823 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Journal.java ##

[GitHub] eolivelli commented on a change in pull request #1317: WIP - BP-14 Force Ledger Bookie side implementation

eolivelli commented on a change in pull request #1317: WIP - BP-14 Force Ledger Bookie side implementation URL: https://github.com/apache/bookkeeper/pull/1317#discussion_r179379018 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SyncCursor.java #

[GitHub] eolivelli commented on issue #1317: WIP - BP-14 Force Ledger Bookie side implementation

eolivelli commented on issue #1317: WIP - BP-14 Force Ledger Bookie side implementation URL: https://github.com/apache/bookkeeper/pull/1317#issuecomment-378855132 This change is not expected to be on 4.7, but it is targeted to 4.8 (hopefully) --

Jenkins build is back to stable : bookkeeper_release_branch #110

See

[GitHub] ivankelly opened a new pull request #1318: MockLedgerHandle duplicates LedgerEntries on read

ivankelly opened a new pull request #1318: MockLedgerHandle duplicates LedgerEntries on read URL: https://github.com/apache/bookkeeper/pull/1318 Previously it was not duplicating, so the underlying ByteBuf was being returned to the client, and this was having its reader index modified,

[GitHub] sijie commented on a change in pull request #1318: MockLedgerHandle duplicates LedgerEntries on read

sijie commented on a change in pull request #1318: MockLedgerHandle duplicates LedgerEntries on read URL: https://github.com/apache/bookkeeper/pull/1318#discussion_r179548960 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/client/MockLedgerHandle.java #

[GitHub] merlimat closed pull request #1318: MockLedgerHandle duplicates LedgerEntries on read

merlimat closed pull request #1318: MockLedgerHandle duplicates LedgerEntries on read URL: https://github.com/apache/bookkeeper/pull/1318 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] sijie opened a new pull request #1319: Issue 1316: A bookie with non-writable dirs should be able to start in readonly mode

sijie opened a new pull request #1319: Issue 1316: A bookie with non-writable dirs should be able to start in readonly mode URL: https://github.com/apache/bookkeeper/pull/1319 Descriptions of the changes in this PR: *Problem* Bookie failed to start when it doesn't have non-wr