ztzg edited a comment on pull request #1644: URL: https://github.com/apache/zookeeper/pull/1644#issuecomment-871197634
@eolivelli, @hanm: Agree that the patch is a bit difficult to digest. It would be great to have another set of eyes, but it takes a substantial amount of effort. On the other hand, it would be better to merge it and deal with the potential consequences of something we would have missed than to have the patch linger for too long. I would merge it into `master`, but I understand that @li4wang would also like to see it in `branch-3.7`, which is a tad more risky. What do you think? (I would be able to take care of the merge later today or tomorrow.) -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
