[GitHub] [pulsar] merlimat commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-07 Thread GitBox
merlimat commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r332197883 ## File path: managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerErrorsTest.java ## @@

[GitHub] [pulsar] merlimat commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-07 Thread GitBox
merlimat commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r332197410 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ## @@ -624,6

[GitHub] [pulsar] merlimat commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-05 Thread GitBox
merlimat commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r331767054 ## File path: managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerErrorsTest.java ## @@

[GitHub] [pulsar] merlimat commented on a change in pull request #5322: Fix bk write failure part 2

2019-10-04 Thread GitBox
merlimat commented on a change in pull request #5322: Fix bk write failure part 2 URL: https://github.com/apache/pulsar/pull/5322#discussion_r331731580 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ## @@ -580,14