[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-1265?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13139186#comment-13139186
 ] 

Hudson commented on ZOOKEEPER-1265:
-----------------------------------

Integrated in ZooKeeper-trunk #1348 (See 
[https://builds.apache.org/job/ZooKeeper-trunk/1348/])
    ZOOKEEPER-1265. Normalize switch cases lists on request types (Thomas Koch 
via phunt)

phunt : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190431
Files : 
* /zookeeper/trunk/CHANGES.txt
* 
/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/CommitProcessor.java
* 
/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/FollowerRequestProcessor.java
* 
/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/ObserverRequestProcessor.java
* 
/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/ReadOnlyRequestProcessor.java

                
> Normalize switch cases lists on request types
> ---------------------------------------------
>
>                 Key: ZOOKEEPER-1265
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1265
>             Project: ZooKeeper
>          Issue Type: Bug
>            Reporter: Thomas Koch
>            Assignee: Thomas Koch
>         Attachments: ZOOKEEPER-1265.patch
>
>
> As discussed on the list, it's probably an error that the 
> ReadOnlyRequestProcessor does not have multi alongside the other write 
> operations.
> Adding check to the lists may not make a difference by now since the ZK 
> client does not expose check as a first level request but only encapsulated 
> inside a multi request. However from a logical view, change belongs in these 
> lists.

--
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

        

Reply via email to