eolivelli commented on issue #2211: bin/bookkeeper shell supports the -conf
URL: https://github.com/apache/bookkeeper/pull/2211#issuecomment-586579005
 
 
   @sijie we have  pushed this PR with 'ignore CI' but this change actually 
broke "bookkeeper shell" and integration tests are no more able to pass.
   
   I am going to push a revert for this fix
   https://github.com/apache/bookkeeper/pull/2265
   
   here you can see that integration tests are passing again.
   
   @SunDapeng1 do you want to restart this work and check why this change does 
not work ?
   In my findings it seems that with this fix there is an endless loop in the 
bash script in case of "bookkeeper shell" command

----------------------------------------------------------------
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