[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-29 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153961253 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-29 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153961253 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-29 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153961253 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-27 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153390983 ## File path:

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-27 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153380301 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SortedLedgerStorage.java

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-27 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153377512 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SyncThread.java ##

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-27 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153377448 ## File path:

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-27 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153354776 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/SyncThread.java ##

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-27 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153354695 ## File path:

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-27 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r153283583 ## File path:

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-21 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r152443892 ## File path:

[GitHub] sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage

2017-11-21 Thread GitBox
sijie commented on a change in pull request #677: Issue 659: Fix Checkpoint logic in SortedLedgerStorage URL: https://github.com/apache/bookkeeper/pull/677#discussion_r152370182 ## File path: