You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zookeeper.apache.org by Jay Shrauner <ja...@fb.com> on 2012/10/13 01:47:49 UTC

Re: Review Request: Multi-thread CommitProcessor

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

(Updated Oct. 12, 2012, 11:47 p.m.)


Review request for zookeeper and Patrick Hunt.


Changes
-------

Address feedback from review--shutdown CommitProcessor if downstream processor throws an exception (preserves previous behavior)


Description
-------

See https://issues.apache.org/jira/browse/ZOOKEEPER-1505


This addresses bug ZOOKEEPER-1505.
    https://issues.apache.org/jira/browse/ZOOKEEPER-1505


Diffs (updated)
-----

  /src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java 1391526 
  /src/java/main/org/apache/zookeeper/server/ServerCnxnFactory.java 1391526 
  /src/java/main/org/apache/zookeeper/server/WorkerService.java PRE-CREATION 
  /src/java/main/org/apache/zookeeper/server/quorum/CommitProcessor.java 1391526 
  /src/java/main/org/apache/zookeeper/server/quorum/Leader.java 1391526 
  /src/java/test/org/apache/zookeeper/server/quorum/CommitProcessorTest.java PRE-CREATION 

Diff: https://reviews.apache.org/r/6260/diff/


Testing
-------


Thanks,

Jay Shrauner