Github user eolivelli commented on the issue:

    https://github.com/apache/bookkeeper/pull/141
  
    @jvrao  we can add a configuration property for "fast shutdown".
    
    IMHO I don't think  that this change really impact bookies even in case of 
graceful shutdown.
    The prev version already called shutdownNow after a timeout.
    With this change a InterruptedException will be more likely to happen, but 
code should already handle this case, so it won't be an issue.
    If we find cases of buggy handling of InterruptedException then we should 
fix them as the InterruptedException can be thrown even without this change.
    
    I have just commented about this new behavior just for completeness and 
because I worried about this case, but from my point of view this is a safe 
change
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to