You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by Thomas Koch <th...@koch.ro> on 2011/11/02 14:57:42 UTC

Review Request: ZOOKEEPER-1246 Dead code in PrepRequestProcessor catch Exception block

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2671/
-----------------------------------------------------------

Review request for zookeeper.


Summary
-------

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-1246.
    https://issues.apache.org/jira/browse/ZOOKEEPER-1246


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


Re: Review Request: ZOOKEEPER-1284 Cleanup minor PrepRequestProcessor issues

Posted by Thomas Koch <th...@koch.ro>.
-----------------------------------------------------------
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