[GitHub] kishorvpatil commented on issue #1125: Issue #1124: Lower memory usage in GarbageCollectionThread while extracting all ledger meta data

2018-02-15 Thread GitBox
kishorvpatil commented on issue #1125: Issue #1124: Lower memory usage in GarbageCollectionThread while extracting all ledger meta data URL: https://github.com/apache/bookkeeper/pull/1125#issuecomment-366120204 @eolivelli and @sijie , I think I addressed all review comments. Please let me

[GitHub] kishorvpatil commented on issue #1125: Issue #1124: Lower memory usage in GarbageCollectionThread while extracting all ledger meta data

2018-02-13 Thread GitBox
kishorvpatil commented on issue #1125: Issue #1124: Lower memory usage in GarbageCollectionThread while extracting all ledger meta data URL: https://github.com/apache/bookkeeper/pull/1125#issuecomment-365485471 @eolivelli Sorry for delay on this. i have fixed the checkstyle issue and also

[GitHub] kishorvpatil commented on issue #1125: Issue #1124: Lower memory usage in GarbageCollectionThread while extracting all ledger meta data

2018-02-13 Thread GitBox
kishorvpatil commented on issue #1125: Issue #1124: Lower memory usage in GarbageCollectionThread while extracting all ledger meta data URL: https://github.com/apache/bookkeeper/pull/1125#issuecomment-365485584 @sijie Removing teardown requires ensure we run this particular test always wit

[GitHub] kishorvpatil commented on issue #1125: Issue #1124: Lower memory usage in GarbageCollectionThread while extracting all ledger meta data

2018-02-12 Thread GitBox
kishorvpatil commented on issue #1125: Issue #1124: Lower memory usage in GarbageCollectionThread while extracting all ledger meta data URL: https://github.com/apache/bookkeeper/pull/1125#issuecomment-365038646 @sijie , The test works with/without `teardown` step at the beginning of the te