[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-24 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r129105288 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-24 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r129104958 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java ## @@ -149,6 +150,8 @

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891404 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieInitializationTest.java ##

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891280 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891319 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891266 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891351 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891390 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieInitializationTest.java ##

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891356 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java ## @@ -146

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891332 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891329 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891401 ## File path: bookkeeper-server/src/test/java/org/apache/bookkeeper/bookie/BookieInitializationTest.java ##

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-22 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891298 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891087 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java ## @@ -149,6 +150,8 @

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891250 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891093 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java ## @@ -748,7 +751,13

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891128 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891221 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891200 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891256 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891187 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891121 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java ## @@ -857,7 +884,13

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891154 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891189 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891227 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2

[GitHub] sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status

2017-07-21 Thread git
sijie commented on a change in pull request #266: Issue 265: Add persistable bookie status URL: https://github.com/apache/bookkeeper/pull/266#discussion_r128891030 ## File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieStatus.java ## @@ -0,0 +1,2