[GitHub] merlimat commented on a change in pull request #1219: Xzy initialize subscription config

2018-02-12 Thread GitBox
merlimat commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167657680 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedger.java ## @@

[GitHub] merlimat commented on a change in pull request #1219: Xzy initialize subscription config

2018-02-12 Thread GitBox
merlimat commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167655263 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedger.java ## @@

[GitHub] merlimat commented on a change in pull request #1219: Xzy initialize subscription config

2018-02-12 Thread GitBox
merlimat commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167657264 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ##

[GitHub] merlimat commented on a change in pull request #1219: Xzy initialize subscription config

2018-02-12 Thread GitBox
merlimat commented on a change in pull request #1219: Xzy initialize subscription config URL: https://github.com/apache/incubator-pulsar/pull/1219#discussion_r167655939 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/api/ConsumerConfiguration.java ##