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 2020/02/15 11:05:22 UTC

[GitHub] [bookkeeper] eolivelli commented on issue #2211: bin/bookkeeper shell supports the -conf

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:
users@infra.apache.org


With regards,
Apache Git Services