You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by GitBox <gi...@apache.org> on 2019/03/27 00:04:17 UTC
[GitHub] [zookeeper] enixon commented on issue #869: ZOOKEEPER-3020: Review
of SyncRequestProcessor
enixon commented on issue #869: ZOOKEEPER-3020: Review of SyncRequestProcessor
URL: https://github.com/apache/zookeeper/pull/869#issuecomment-476903563
One way to proceed might be to separate the logic changes into separate commits in a single PR. At a brief glance, it doesn't look like important logic is changed (and I like the cleaner end result!) but its hard to confirm that with all the changes overlapping each other.
----------------------------------------------------------------
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