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: [email protected] With regards, Apache Git Services
