You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2018/05/21 11:09:42 UTC

[GitHub] sijie opened a new pull request #1416: Provide zookeeper startup script

sijie opened a new pull request #1416: Provide zookeeper startup script
URL: https://github.com/apache/bookkeeper/pull/1416
 
 
   Descriptions of the changes in this PR:
   
   *Motivation*
   
   ZooKeeper is a dependency of bookkeeper shipped along with bookkeeper dist package. However we never provide any script to startup zookeeper in the bookkeeper binary package and never test the zookeeper version shipped along with bookkeeper 
   
   *Solution*
   
   - add zookeeper in `bin/bookeeper` scripts to start zookeeper using bookkeeper scripts
   - refactor bookkeeper docker scripts to start bookie/zookeeper/standalone
   - use zk adminPort for health check, since 4letters commands are explicitly disabled at 3.5
   
   *Tests*
   
   Existing integration tests test the docker scripts and the bash script changes to start zookeeper.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services