[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-366421598 all CI passed (expect integration tests). because this PR doesn't contain the fix in #1158 . sin

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-366421604 IGNORE CI This is an automated

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-366297798 retest this please This is an a

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-366286182 retest this please This is an a

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-365681563 retest this please This is an a

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-365681541 ping @eolivelli This is an aut

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-364621459 @eolivelli since you were involved in this pr, can you review this PR again? I think @jiazhai al

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-362752654 @eolivelli can you review this again? @jiazhai already addressed your comment. @revans2 @

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-362127475 @govind-menon @merlimat can you guys review this cherry-pick? --

[GitHub] sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments

sijie commented on issue #1085: [Merge Yahoo repo]: YBK-160: Doing distributed random verification of ledger fragments URL: https://github.com/apache/bookkeeper/pull/1085#issuecomment-361693650 @govind-menon can you review this cherry-pick?