[GitHub] [zookeeper] anmolnar commented on issue #838: ZOOKEEPER-2168 Add C APIs for new createContainer Methods

2019-03-18 Thread GitBox
anmolnar commented on issue #838: ZOOKEEPER-2168 Add C APIs for new createContainer Methods URL: https://github.com/apache/zookeeper/pull/838#issuecomment-474149379 Merged to 3.5 and master branches. Thanks @meszibalu ! This

[GitHub] [zookeeper] anmolnar commented on issue #838: ZOOKEEPER-2168 Add C APIs for new createContainer Methods

2019-03-14 Thread GitBox
anmolnar commented on issue #838: ZOOKEEPER-2168 Add C APIs for new createContainer Methods URL: https://github.com/apache/zookeeper/pull/838#issuecomment-473108515 @meszibalu I tend to accept this patch with that unit test. Seems like there's literally nothing more we can do about it.

[GitHub] [zookeeper] anmolnar commented on issue #838: ZOOKEEPER-2168 Add C APIs for new createContainer Methods

2019-03-11 Thread GitBox
anmolnar commented on issue #838: ZOOKEEPER-2168 Add C APIs for new createContainer Methods URL: https://github.com/apache/zookeeper/pull/838#issuecomment-471754749 @meszibalu Thanks for looking into. @Randgalt Is that accurate? Client is unable to identify a container node from the