[GitHub] sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl`

2018-02-16 Thread GitBox
sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl` URL: https://github.com/apache/bookkeeper/pull/1127#issuecomment-366301360 merged to latest master (so integration tests can actually run) This

[GitHub] sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl`

2018-02-16 Thread GitBox
sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl` URL: https://github.com/apache/bookkeeper/pull/1127#issuecomment-366297053 retest this please This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl`

2018-02-16 Thread GitBox
sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl` URL: https://github.com/apache/bookkeeper/pull/1127#issuecomment-366296956 @eolivelli I will merge it after CI passed. I need this for subsequent changes.

[GitHub] sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl`

2018-02-15 Thread GitBox
sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl` URL: https://github.com/apache/bookkeeper/pull/1127#issuecomment-366153789 retest this please This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl`

2018-02-12 Thread GitBox
sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl` URL: https://github.com/apache/bookkeeper/pull/1127#issuecomment-364982594 ping @jvrao @eolivelli , please review so that I can move forward with other changes.

[GitHub] sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl`

2018-02-09 Thread GitBox
sijie commented on issue #1127: BP-29 (task 1) : Introduce `metadataServiceUrl` URL: https://github.com/apache/bookkeeper/pull/1127#issuecomment-364619629 @eolivelli @jvrao I already addressed your comments. please review this again.