[GitHub] reddycharan commented on a change in pull request #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage

2018-03-07 Thread GitBox
reddycharan commented on a change in pull request #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/1225#discussion_r173080583 ## File path:

[GitHub] reddycharan opened a new pull request #1236: Issue #570: make changes to SyncThread/checkpoint logic.

2018-03-07 Thread GitBox
reddycharan opened a new pull request #1236: Issue #570: make changes to SyncThread/checkpoint logic. URL: https://github.com/apache/bookkeeper/pull/1236 Descriptions of the changes in this PR: make changes to SyncThread/checkpoint logic, so that incase of entrylogperledger,

[GitHub] kishorekasi commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
kishorekasi commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r173033139 ## File path:

[GitHub] reddycharan commented on a change in pull request #1228: Issue #570: Move logic of unpersistedbytes to bufferedchannel

2018-03-07 Thread GitBox
reddycharan commented on a change in pull request #1228: Issue #570: Move logic of unpersistedbytes to bufferedchannel URL: https://github.com/apache/bookkeeper/pull/1228#discussion_r173007997 ## File path:

[GitHub] reddycharan commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
reddycharan commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r173003469 ## File path:

[GitHub] reddycharan commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
reddycharan commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r173003469 ## File path:

Jenkins build is back to stable : bookkeeper_postcommit_master_java9 #63

2018-03-07 Thread Apache Jenkins Server
See

[GitHub] ArvinDevel commented on a change in pull request #832: Issue 620: Close the fileChannels for read when they are idle

2018-03-07 Thread GitBox
ArvinDevel commented on a change in pull request #832: Issue 620: Close the fileChannels for read when they are idle URL: https://github.com/apache/bookkeeper/pull/832#discussion_r172843523 ## File path:

[GitHub] ivankelly commented on issue #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
ivankelly commented on issue #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#issuecomment-371115598 @reddycharan > don't think so. is there any todo list to get one? You can link your apache and github

[GitHub] ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r172818962 ## File path:

[GitHub] ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r172818263 ## File path:

[GitHub] ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r172818307 ## File path:

[GitHub] ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r172811485 ## File path:

[GitHub] ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r172810937 ## File path:

[GitHub] ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop

2018-03-07 Thread GitBox
ivankelly commented on a change in pull request #1233: Reduce running time for testLedgerCreateAdvWithLedgerIdInLoop URL: https://github.com/apache/bookkeeper/pull/1233#discussion_r172810914 ## File path:

[GitHub] ivankelly commented on issue #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage

2018-03-07 Thread GitBox
ivankelly commented on issue #1225: Issue #570: getting rid of unnecessary synchronization in InterleavedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/1225#issuecomment-371106310 @reddycharan number of tasks should be fine. I'd just remove the timeout. Up to you though.

[GitHub] sijie opened a new pull request #1235: Move dependency version management to root pom file

2018-03-07 Thread GitBox
sijie opened a new pull request #1235: Move dependency version management to root pom file URL: https://github.com/apache/bookkeeper/pull/1235 Descriptions of the changes in this PR: *Motivation* To have a clear dependency management at bookkeeper, avoid unnecessary version