[ https://issues.apache.org/jira/browse/ZOOKEEPER-1284?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13143027#comment-13143027 ]
jirapos...@reviews.apache.org commented on ZOOKEEPER-1284: ---------------------------------------------------------- ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2671/ ----------------------------------------------------------- (Updated 2011-11-03 10:12:51.768461) Review request for zookeeper. Changes ------- Changed this review request to relate to issue ZOOKEEPER-1284 Summary (updated) ------- This is _not_ a diff againt current trunk but against the trunk _before_ the first version of the ZK-1246 patch got committed. This addresses bug ZOOKEEPER-1284. https://issues.apache.org/jira/browse/ZOOKEEPER-1284 Diffs ----- CHANGES.txt 8ed2bc2 src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java 1a80d74 src/java/test/org/apache/zookeeper/server/PrepRequestProcessorTest.java PRE-CREATION Diff: https://reviews.apache.org/r/2671/diff Testing ------- Thanks, Thomas > Cleanup minor PrepRequestProcessor issues > ----------------------------------------- > > Key: ZOOKEEPER-1284 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1284 > Project: ZooKeeper > Issue Type: Sub-task > Reporter: Thomas Koch > Assignee: Thomas Koch > Priority: Minor > Attachments: ZOOKEEPER-1284.patch > > > Instead of having if statements in every switch case in pRequest2Txn, it is > possible to have only one if statement before the switch case in pRequest. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira