[GitHub] sijie commented on issue #1325: Update missing configuration settings & missing shell commands

2018-04-10 Thread GitBox
sijie commented on issue #1325: Update missing configuration settings & missing shell commands URL: https://github.com/apache/bookkeeper/pull/1325#issuecomment-380033520 ignore ci, since this change only touch configuration file and website, and travis CI passed.

[GitHub] sijie commented on issue #1325: Update missing configuration settings & missing shell commands

2018-04-10 Thread GitBox
sijie commented on issue #1325: Update missing configuration settings & missing shell commands URL: https://github.com/apache/bookkeeper/pull/1325#issuecomment-380033571 IGNORE CI This is an automated message from the Apache

[GitHub] sijie closed pull request #1325: Update missing configuration settings & missing shell commands

2018-04-10 Thread GitBox
sijie closed pull request #1325: Update missing configuration settings & missing shell commands URL: https://github.com/apache/bookkeeper/pull/1325 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] eolivelli commented on issue #1327: Update News after release of 4.6.2

2018-04-10 Thread GitBox
eolivelli commented on issue #1327: Update News after release of 4.6.2 URL: https://github.com/apache/bookkeeper/pull/1327#issuecomment-380040916 @sijie could you please merge this one ? This is an automated message from the

[GitHub] eolivelli opened a new pull request #1327: Update News after release of 4.6.2

2018-04-10 Thread GitBox
eolivelli opened a new pull request #1327: Update News after release of 4.6.2 URL: https://github.com/apache/bookkeeper/pull/1327 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] eolivelli commented on issue #1327: Update News after release of 4.6.2

2018-04-10 Thread GitBox
eolivelli commented on issue #1327: Update News after release of 4.6.2 URL: https://github.com/apache/bookkeeper/pull/1327#issuecomment-380038769 @sijie I did the change very quickly, I must have missed one key :-) thank you

[GitHub] eolivelli closed pull request #1327: Update News after release of 4.6.2

2018-04-10 Thread GitBox
eolivelli closed pull request #1327: Update News after release of 4.6.2 URL: https://github.com/apache/bookkeeper/pull/1327 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

Jenkins build became unstable: bookkeeper_postcommit_master_java9 #97

2018-04-10 Thread Apache Jenkins Server
See

Jenkins build is back to stable : bookkeeper_release_branch #115

2018-04-10 Thread Apache Jenkins Server
See

Jenkins build is back to stable : bookkeeper_postcommit_master_java8 #97

2018-04-10 Thread Apache Jenkins Server
See

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180514235 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -788,89

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180515728 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -788,89

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180524406 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java ## @@

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180514001 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -788,89

[GitHub] sijie commented on issue #914: Document bookkeeper shell command changes

2018-04-10 Thread GitBox
sijie commented on issue #914: Document bookkeeper shell command changes URL: https://github.com/apache/bookkeeper/issues/914#issuecomment-380217833 This is done by #1325 This is an automated message from the Apache Git

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180507966 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180510817 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -880,39

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180510213 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -788,89

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180507116 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180516551 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -788,89

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180508758 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180525552 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/CreateNewLogTest.java ## @@

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180505515 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -120,21

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180520311 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java ## @@

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180520230 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/ReadOnlyEntryLogger.java ## @@

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180516776 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -788,89

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180517926 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -788,89

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180548567 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180505355 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -84,6

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180508334 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
sijie commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180505208 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@ -92,13

[GitHub] sijie closed issue #914: Document bookkeeper shell command changes

2018-04-10 Thread GitBox
sijie closed issue #914: Document bookkeeper shell command changes URL: https://github.com/apache/bookkeeper/issues/914 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] reddycharan commented on issue #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on issue #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#issuecomment-380157141 Hey @sijie I think I addressed all of your comments and tests are looking good.

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180583624 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180588670 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180590406 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180581487 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/ReadOnlyEntryLogger.java

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180583759 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180584833 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180593414 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] sijie opened a new pull request #1328: Create CI jobs for branch-4.7

2018-04-10 Thread GitBox
sijie opened a new pull request #1328: Create CI jobs for branch-4.7 URL: https://github.com/apache/bookkeeper/pull/1328 Descriptions of the changes in this PR: We need to CI jobs running for subsequent minor releases on branch-4.7.

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180565394 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/CreateNewLogTest.java ##

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180555218 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180555218 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java

[GitHub] sijie commented on issue #1328: Create CI jobs for branch-4.7

2018-04-10 Thread GitBox
sijie commented on issue #1328: Create CI jobs for branch-4.7 URL: https://github.com/apache/bookkeeper/pull/1328#issuecomment-380334666 IGNORE CI This is an automated message from the Apache Git Service. To respond to the

[GitHub] sijie opened a new pull request #1329: (WIP) Issue 1246: Scripting the whole release procedure

2018-04-10 Thread GitBox
sijie opened a new pull request #1329: (WIP) Issue 1246: Scripting the whole release procedure URL: https://github.com/apache/bookkeeper/pull/1329 Descriptions of the changes in this PR: *Motivation* Releasing bookkeeper becomes tricky after introducing `circe-checksum`,

[GitHub] sijie closed pull request #1328: Create CI jobs for branch-4.7

2018-04-10 Thread GitBox
sijie closed pull request #1328: Create CI jobs for branch-4.7 URL: https://github.com/apache/bookkeeper/pull/1328 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

[GitHub] reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager.

2018-04-10 Thread GitBox
reddycharan commented on a change in pull request #1281: Issue #570: Introducing EntryLogManager. URL: https://github.com/apache/bookkeeper/pull/1281#discussion_r180642016 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java ## @@

[GitHub] sijie opened a new pull request #1325: Update missing configuration settings & missing shell commands

2018-04-10 Thread GitBox
sijie opened a new pull request #1325: Update missing configuration settings & missing shell commands URL: https://github.com/apache/bookkeeper/pull/1325 Descriptions of the changes in this PR: Update missing configuration settings & missing shell commands for preparing 4.7.0

[GitHub] sijie commented on issue #1325: Update missing configuration settings & missing shell commands

2018-04-10 Thread GitBox
sijie commented on issue #1325: Update missing configuration settings & missing shell commands URL: https://github.com/apache/bookkeeper/pull/1325#issuecomment-379995170 This PR was based on the change at #1324 . 4d6e1ab is the change for review.

[GitHub] sijie closed pull request #1324: Remove WriteFlags from public API

2018-04-10 Thread GitBox
sijie closed pull request #1324: Remove WriteFlags from public API URL: https://github.com/apache/bookkeeper/pull/1324 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] sijie opened a new pull request #1326: Prepare 4.7.0 release

2018-04-10 Thread GitBox
sijie opened a new pull request #1326: Prepare 4.7.0 release URL: https://github.com/apache/bookkeeper/pull/1326 Descriptions of the changes in this PR: - copy the current `docs/latest` as `docs/4.7.0` - bump latest development release to 4.8.0 SNAPSHOT release