eolivelli edited a comment on issue #2112: Issue #2106: Update ZookKeeper 
dependency to 3.5.5
URL: https://github.com/apache/bookkeeper/pull/2112#issuecomment-531576441
 
 
   @dmercuriali I guess this is the problem
   
   here we are checking the result code from zk shell invoked thru BookKeeper 
shell
   
https://github.com/apache/bookkeeper/blob/4c6cf097d9b823c8e839e176a06266a58bcb7bf5/docker/scripts/init_bookie.sh#L46
   
   /opt/bookkeeper/bin/bookkeeper org.apache.zookeeper.ZooKeeperMain -server 
${BK_zkServers} stat ${BK_STREAM_STORAGE_ROOT_PATH}
   
   Maybe with new ZK the ZooKeeperMain class does not return the expected 
return code.
   
   Please check as soon as possible, a new release of ZK 3.5.6 is going to be 
released soon, if we have a problem we should fix in ZK as soon as possible
   
   
   I suggest you to print the value of the return code
   

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


With regards,
Apache Git Services

Reply via email to