eolivelli commented on issue #2206: Remove lock from URManager’s state only if lock znode deletion has succeeded URL: https://github.com/apache/bookkeeper/pull/2206#issuecomment-554674537 Yes, we can have two classes, they are doing different things. Having a single MockZookeeperClient can complicate things and maybe affect each other test suites. So I'd stay as Charan is proposing
---------------------------------------------------------------- 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
