[GitHub] reddycharan opened a new pull request #1199: Provide default values for LedgerIdFormatter and EntryFormatter

2018-02-22 Thread GitBox
reddycharan opened a new pull request #1199: Provide default values for LedgerIdFormatter and EntryFormatter URL: https://github.com/apache/bookkeeper/pull/1199 Descriptions of the changes in this PR: Provide default values for LedgerIdFormatter and EntryFormatter.

[GitHub] sijie closed issue #1107: Use loopback network interface for testcases.

2018-02-22 Thread GitBox
sijie closed issue #1107: Use loopback network interface for testcases. URL: https://github.com/apache/bookkeeper/issues/1107 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] sijie commented on issue #1107: Use loopback network interface for testcases.

2018-02-22 Thread GitBox
sijie commented on issue #1107: Use loopback network interface for testcases. URL: https://github.com/apache/bookkeeper/issues/1107#issuecomment-367597153 This is fixed by #1097 This is an automated message from the Apache

[GitHub] sijie closed issue #1030: BC test cleanup

2018-02-22 Thread GitBox
sijie closed issue #1030: BC test cleanup URL: https://github.com/apache/bookkeeper/issues/1030 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above

[GitHub] ivankelly commented on a change in pull request #898: Bookies should prioritize recovery reads/writes

2018-02-22 Thread GitBox
ivankelly commented on a change in pull request #898: Bookies should prioritize recovery reads/writes URL: https://github.com/apache/bookkeeper/pull/898#discussion_r169886152 ## File path: bookkeeper-proto/src/main/proto/BookkeeperProtocol.proto ## @@ -90,8 +90,9 @@

[GitHub] sijie commented on issue #1101: Deprecate FlatLedgerManager

2018-02-22 Thread GitBox
sijie commented on issue #1101: Deprecate FlatLedgerManager URL: https://github.com/apache/bookkeeper/issues/1101#issuecomment-367597535 This has been fixed by #1115 This is an automated message from the Apache Git Service.

[GitHub] sijie closed issue #1101: Deprecate FlatLedgerManager

2018-02-22 Thread GitBox
sijie closed issue #1101: Deprecate FlatLedgerManager URL: https://github.com/apache/bookkeeper/issues/1101 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] sijie commented on issue #881: Separate initialize from format

2018-02-22 Thread GitBox
sijie commented on issue #881: Separate initialize from format URL: https://github.com/apache/bookkeeper/issues/881#issuecomment-367598891 The `metaformat` command has been implemented with two commands `initnewcluster ` and `nukeexistingcluster` in #978. we should update the docker file

[GitHub] sijie closed issue #656: Enable tests running with advertised address "127.0.0.1"

2018-02-22 Thread GitBox
sijie closed issue #656: Enable tests running with advertised address "127.0.0.1" URL: https://github.com/apache/bookkeeper/issues/656 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] sijie commented on issue #656: Enable tests running with advertised address "127.0.0.1"

2018-02-22 Thread GitBox
sijie commented on issue #656: Enable tests running with advertised address "127.0.0.1" URL: https://github.com/apache/bookkeeper/issues/656#issuecomment-36769 This has been fixed by #1097 This is an automated message

[GitHub] ivankelly commented on a change in pull request #898: Bookies should prioritize recovery reads/writes

2018-02-22 Thread GitBox
ivankelly commented on a change in pull request #898: Bookies should prioritize recovery reads/writes URL: https://github.com/apache/bookkeeper/pull/898#discussion_r169886394 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieRequestProcessor.java

[GitHub] sijie closed issue #1031: Only allow merging with green CI results

2018-02-22 Thread GitBox
sijie closed issue #1031: Only allow merging with green CI results URL: https://github.com/apache/bookkeeper/issues/1031 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] sijie commented on issue #1031: Only allow merging with green CI results

2018-02-22 Thread GitBox
sijie commented on issue #1031: Only allow merging with green CI results URL: https://github.com/apache/bookkeeper/issues/1031#issuecomment-367597813 This has been fixed by #1145 This is an automated message from the Apache

[GitHub] ivankelly commented on a change in pull request #898: Bookies should prioritize recovery reads/writes

2018-02-22 Thread GitBox
ivankelly commented on a change in pull request #898: Bookies should prioritize recovery reads/writes URL: https://github.com/apache/bookkeeper/pull/898#discussion_r169888117 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieRequestProcessor.java

[GitHub] hrsakai commented on issue #1197: A lot of underreplicated ledgers occur when periodic check run

2018-02-22 Thread GitBox
hrsakai commented on issue #1197: A lot of underreplicated ledgers occur when periodic check run URL: https://github.com/apache/bookkeeper/issues/1197#issuecomment-367613810 @sijie These are auditor logs when a lot of ledgers were marked as underreplicated.

[GitHub] sijie closed pull request #1198: Update committers list

2018-02-22 Thread GitBox
sijie closed pull request #1198: Update committers list URL: https://github.com/apache/bookkeeper/pull/1198 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 request (from

[GitHub] eolivelli commented on issue #1193: Backup/Restore Mechanism for BookKeeper

2018-02-22 Thread GitBox
eolivelli commented on issue #1193: Backup/Restore Mechanism for BookKeeper URL: https://github.com/apache/bookkeeper/issues/1193#issuecomment-367619550 I did ask for such kind of feature last year. This is an automated

[GitHub] ivankelly commented on issue #1192: BP-29 (task 3): use metadata service uri for constructing registration client

2018-02-22 Thread GitBox
ivankelly commented on issue #1192: BP-29 (task 3): use metadata service uri for constructing registration client URL: https://github.com/apache/bookkeeper/pull/1192#issuecomment-367648661 The integration test failure looks legit

Jenkins build became unstable: bookkeeper_release_branch #67

2018-02-22 Thread Apache Jenkins Server
See

[GitHub] ivankelly commented on issue #898: Bookies should prioritize recovery reads/writes

2018-02-22 Thread GitBox
ivankelly commented on issue #898: Bookies should prioritize recovery reads/writes URL: https://github.com/apache/bookkeeper/pull/898#issuecomment-367652737 Yahoo uses the V2 protocol, so the change should be compatible (the whole BC test changes are to ensure this, there's a test with

[GitHub] massakam opened a new issue #1200: Status of some ledgers have remained IN_RECOVERY for long

2018-02-22 Thread GitBox
massakam opened a new issue #1200: Status of some ledgers have remained IN_RECOVERY for long URL: https://github.com/apache/bookkeeper/issues/1200 **BUG REPORT** What did you do? We are using BookKeeper with [Pulsar](https://github.com/apache/incubator-pulsar).

[GitHub] massakam commented on issue #1200: Status of some ledgers have remained IN_RECOVERY for long

2018-02-22 Thread GitBox
massakam commented on issue #1200: Status of some ledgers have remained IN_RECOVERY for long URL: https://github.com/apache/bookkeeper/issues/1200#issuecomment-367890734 This is dump of thread which is consuming high CPU. https://gist.github.com/massakam/df6a7b1fbecabd8de7a38ff368c4cb06

Jenkins build is still unstable: bookkeeper_release_nightly_snapshot #67

2018-02-22 Thread Apache Jenkins Server
See