[GitHub] sijie commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
sijie commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394630577 rerun integration tests This is an automated message from the Apache Git Service. To

[GitHub] sijie commented on issue #1479: Fix issues for cherry-picks from master to branch-4.7

2018-06-05 Thread GitBox
sijie commented on issue #1479: Fix issues for cherry-picks from master to branch-4.7 URL: https://github.com/apache/bookkeeper/pull/1479#issuecomment-394601130 merged as c9b58b2 in brach-4.7. This is an automated message

[GitHub] sijie closed pull request #1479: Fix issues for cherry-picks from master to branch-4.7

2018-06-05 Thread GitBox
sijie closed pull request #1479: Fix issues for cherry-picks from master to branch-4.7 URL: https://github.com/apache/bookkeeper/pull/1479 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 #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
sijie commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394590813 rerun integration tests This is an automated message from the Apache Git Service. To respond

[GitHub] sijie commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
sijie commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394630577 rerun integration tests This is an automated message from the Apache Git Service. To

[GitHub] sijie commented on issue #1479: Fix issues for cherry-picks from master to branch-4.7

2018-06-05 Thread GitBox
sijie commented on issue #1479: Fix issues for cherry-picks from master to branch-4.7 URL: https://github.com/apache/bookkeeper/pull/1479#issuecomment-394600381 IGNORE CI This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1480: Throttle integration tests to run 1 build per host

2018-06-05 Thread GitBox
sijie commented on issue #1480: Throttle integration tests to run 1 build per host URL: https://github.com/apache/bookkeeper/pull/1480#issuecomment-394600122 rerun integration tests This is an automated message from the

[GitHub] sijie commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
sijie commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394596299 rerun integration tests This is an automated message from the Apache Git Service. To respond

[GitHub] sijie opened a new pull request #1481: Improve precommit build time

2018-06-05 Thread GitBox
sijie opened a new pull request #1481: Improve precommit build time URL: https://github.com/apache/bookkeeper/pull/1481 Descriptions of the changes in this PR: *Motivation* The build/test time of `bookkeeper-server` is almost close to or more than 1 hr. *Solution*

[GitHub] sijie closed pull request #1480: Throttle integration tests to run 1 build per host

2018-06-05 Thread GitBox
sijie closed pull request #1480: Throttle integration tests to run 1 build per host URL: https://github.com/apache/bookkeeper/pull/1480 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 commented on issue #1476: recycle already.

2018-06-05 Thread GitBox
sijie commented on issue #1476: recycle already. URL: https://github.com/apache/bookkeeper/issues/1476#issuecomment-394640631 @infodog can you describe more about this issue? - when is this exception thrown? during reading or during shutdown. I was looking into the code and couldn't

[GitHub] infodog commented on issue #1476: recycle already.

2018-06-05 Thread GitBox
infodog commented on issue #1476: recycle already. URL: https://github.com/apache/bookkeeper/issues/1476#issuecomment-394656506 @sijie my situation is when one process reading from the log, and one process writing to the log, the writer is fast. when this exception happened, the reader

[GitHub] infodog commented on issue #1476: recycle already.

2018-06-05 Thread GitBox
infodog commented on issue #1476: recycle already. URL: https://github.com/apache/bookkeeper/issues/1476#issuecomment-394689982 @sijie maybe this issue is like #1229 ? Similar reason? This is an automated message from the

[GitHub] infodog commented on issue #1476: recycle already.

2018-06-05 Thread GitBox
infodog commented on issue #1476: recycle already. URL: https://github.com/apache/bookkeeper/issues/1476#issuecomment-394656506 @sijie my situation is when one process reading from the log, and one process writing to the log, the writer is fast. when this exception happened, the reader

Build failed in Jenkins: bookkeeper_release_branch_47_integrationtests #47

2018-06-05 Thread Apache Jenkins Server
See Changes: [sijie] Use unpooled buffers in BufferedChannel [sijie] [TABLE SERVICE] Dlog based checkpoint store [sijie] [TABLE SERVICE] apply backoff policy to rpc requests if

Jenkins build became unstable: bookkeeper_release_branch_46 #56

2018-06-05 Thread Apache Jenkins Server
See

Jenkins build is back to normal : bookkeeper_release_nightly_snapshot #22

2018-06-05 Thread Apache Jenkins Server
See

Build failed in Jenkins: bookkeeper_release_branch_47_java8 #57

2018-06-05 Thread Apache Jenkins Server
See Changes: [sijie] Use unpooled buffers in BufferedChannel [sijie] [TABLE SERVICE] Dlog based checkpoint store [sijie] [TABLE SERVICE] apply backoff policy to rpc requests if storage [sijie]

[GitHub] eolivelli commented on issue #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
eolivelli commented on issue #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#issuecomment-394740832 retest this please This is an automated message from the Apache

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394812136 rebuild This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394812136 rebuild This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
sijie commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394813873 retest this please This is an automated message from the Apache Git Service.

[GitHub] asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394822963 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/bookkeeper_precommit_tls_tests/3/

[GitHub] asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394823153 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/bookkeeper_precommit_bookie_tests/3/

[GitHub] asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394822962 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/bookkeeper_precommit_replication_tests/3/

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394812032 run integration tests This is an automated message from the

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394816208 Refer to this link for build results (access rights to CI server needed):

Jenkins build is back to normal : bookkeeper-seed #208

2018-06-05 Thread Apache Jenkins Server
See

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394811278 @reddycharan I think so. let me try to retest integration tests again.

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394816792 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: bookkeeper-seed #205

2018-06-05 Thread Apache Jenkins Server
See Changes: [sijie] Throttle integration tests to run 1 build per host [sijie] Improve precommit build time -- Started by user sijie [EnvInject] - Loading node

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394816805 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394816567 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394819355 Refer to this link for build results (access rights to CI server needed):

[GitHub] reddycharan commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
reddycharan commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394786755 @sijie i think this commit should be good to go. This is

[GitHub] sijie commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
sijie commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394803323 rick it This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394811416 run tests This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394812821 rebuild java8 This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394812821 rebuild java8 This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394818078 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394819243 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394819310 Refer to this link for build results (access rights to CI server needed):

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r191294774 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieClient.java ## @@

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193184413 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieDeferredSyncTest.java

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193184878 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieDeferredSyncTest.java

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193181776 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerHandle.java ## @@

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193179387 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/ForceLedgerOp.java ## @@

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193183088 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieDeferredSyncTest.java

[GitHub] reddycharan commented on issue #1460: Metrics for EntryLogManagerForEntryLogPerLedger.

2018-06-05 Thread GitBox
reddycharan commented on issue #1460: Metrics for EntryLogManagerForEntryLogPerLedger. URL: https://github.com/apache/bookkeeper/pull/1460#issuecomment-394786491 @sijie i fixed all the compile time / checkstyle issues, I guess this should be good to go.

[GitHub] sijie commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
sijie commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394803236 It seems integration tests keep timed out on H30 node. This is an automated message from

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394817685 Refer to this link for build results (access rights to CI server needed):

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193178856 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/ForceLedgerOp.java ## @@

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193180806 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerHandle.java ## @@

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193185487 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieDeferredSyncTest.java

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193185159 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieDeferredSyncTest.java

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394833659 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394833819 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394838145 Refer to this link for build results (access rights to CI server needed):

[GitHub] sijie commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
sijie commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394840273 Closed this PR for now. will reopen when I fixed the jenkins. This is an

[GitHub] sijie opened a new pull request #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
sijie opened a new pull request #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425 Descriptions of the changes in this PR: This PR adds smoke tests for dlog library to cover basic functionalities like basic write/read,

Jenkins build is back to normal : bookkeeper-seed #210

2018-06-05 Thread Apache Jenkins Server
See

[GitHub] sijie commented on issue #1476: recycle already.

2018-06-05 Thread GitBox
sijie commented on issue #1476: recycle already. URL: https://github.com/apache/bookkeeper/issues/1476#issuecomment-394846173 @infodog - I think it is similar, it looks like a double-release issue. will take a closer look.

[GitHub] sijie commented on issue #1460: Metrics for EntryLogManagerForEntryLogPerLedger.

2018-06-05 Thread GitBox
sijie commented on issue #1460: Metrics for EntryLogManagerForEntryLogPerLedger. URL: https://github.com/apache/bookkeeper/pull/1460#issuecomment-394869981 run pr validation This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394825431 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394825436 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394827291 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/bookkeeper-seed/209/

Build failed in Jenkins: bookkeeper_release_branch_46 #57

2018-06-05 Thread Apache Jenkins Server
See -- [...truncated 394.74 KB...] 2018-06-05T19:17:06.498 [INFO] Downloading:

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394835729 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394837473 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394837349 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394837369 Refer to this link for build results (access rights to CI server needed):

[GitHub] sijie closed pull request #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
sijie closed pull request #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425 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

[GitHub] sijie closed pull request #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
sijie closed pull request #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425 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

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394838834 @reddycharan feel free to ignore the jenkins comments. I am fixing jenkins now.

[GitHub] sijie commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
sijie commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394845101 (I needed this PR to be included in 4.7.1, in order to align the rocksdb version used for pulsar 2.1)

[GitHub] sijie commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
sijie commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394844866 not sure why the integration tests keep failing. there is no a successful run for this PR.

[GitHub] dlg99 commented on issue #1468: Issue #1467: build failing with MavenReportException: Error while generating Javadoc.

2018-06-05 Thread GitBox
dlg99 commented on issue #1468: Issue #1467: build failing with MavenReportException: Error while generating Javadoc. URL: https://github.com/apache/bookkeeper/pull/1468#issuecomment-394846905 @sijie my apologies, missed that. will do next time.

[GitHub] sijie opened a new pull request #1482: Remove phrase trigger on post commit jobs

2018-06-05 Thread GitBox
sijie opened a new pull request #1482: Remove phrase trigger on post commit jobs URL: https://github.com/apache/bookkeeper/pull/1482 Descriptions of the changes in this PR: *Problem* After enabling phrase trigger on precommit jobs to allow rerun individual precommit jobs,

[GitHub] sijie commented on issue #1460: Metrics for EntryLogManagerForEntryLogPerLedger.

2018-06-05 Thread GitBox
sijie commented on issue #1460: Metrics for EntryLogManagerForEntryLogPerLedger. URL: https://github.com/apache/bookkeeper/pull/1460#issuecomment-394870628 actually all CI passed. the failed check is a dangling check. ignore ci to merge.

[GitHub] sijie commented on issue #1460: Metrics for EntryLogManagerForEntryLogPerLedger.

2018-06-05 Thread GitBox
sijie commented on issue #1460: Metrics for EntryLogManagerForEntryLogPerLedger. URL: https://github.com/apache/bookkeeper/pull/1460#issuecomment-394870665 IGNORE CI This is an automated message from the Apache Git Service.

[GitHub] asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394824757 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/bookkeeper_precommit_remaining_tests/3/

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394826364 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394831562 Refer to this link for build results (access rights to CI server needed):

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193184253 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieDeferredSyncTest.java

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193178082 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/DistributionSchedule.java

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193180876 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerHandle.java ## @@

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394826842 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394827991 Refer to this link for build results (access rights to CI server needed):

Build failed in Jenkins: bookkeeper_postcommit_validation_master #4

2018-06-05 Thread Apache Jenkins Server
See -- [...truncated 1.23 MB...] 2018-06-05T19:32:24.171 [INFO] Exclude: **/*.iws 2018-06-05T19:32:24.171 [INFO] Exclude: **/*.ipr 2018-06-05T19:32:24.171 [INFO]

Build failed in Jenkins: bookkeeper_release_branch_46 #58

2018-06-05 Thread Apache Jenkins Server
See -- [...truncated 389.16 KB...] 2018-06-05T20:47:57.279 [INFO] Downloading:

[GitHub] sijie closed pull request #1478: Update the new bookkeeper tools to use the new framework

2018-06-05 Thread GitBox
sijie closed pull request #1478: Update the new bookkeeper tools to use the new framework URL: https://github.com/apache/bookkeeper/pull/1478 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 commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394840655 All tests CI passed. ignored other CI to allow merge. This is

[GitHub] sijie closed pull request #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie closed pull request #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465 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] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394844784 Refer to this link for build results (access rights to CI server needed):

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193184113 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieDeferredSyncTest.java

[GitHub] sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation

2018-06-05 Thread GitBox
sijie commented on a change in pull request #1436: BP-14 force() API - client side implementation URL: https://github.com/apache/bookkeeper/pull/1436#discussion_r193183906 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieDeferredSyncTest.java

[GitHub] asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
asfgit commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394825870 Refer to this link for build results (access rights to CI server needed):

[GitHub] asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1

2018-06-05 Thread GitBox
asfgit commented on issue #1466: Upgrade RocksDB to 5.13.1 URL: https://github.com/apache/bookkeeper/pull/1466#issuecomment-394827751 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/bookkeeper_precommit_client_tests/3/

[GitHub] asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library

2018-06-05 Thread GitBox
asfgit commented on issue #1425: [dlog] Add smoke tests for dlog library URL: https://github.com/apache/bookkeeper/pull/1425#issuecomment-394829933 Refer to this link for build results (access rights to CI server needed):

[GitHub] sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache.

2018-06-05 Thread GitBox
sijie commented on issue #1465: Append ledgersMap when entrylog is removed from cache. URL: https://github.com/apache/bookkeeper/pull/1465#issuecomment-394841013 IGNORE CI This is an automated message from the Apache Git

  1   2   >