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

Hudson commented on ZOOKEEPER-1267:
-----------------------------------

Integrated in ZooKeeper-trunk #1348 (See 
[https://builds.apache.org/job/ZooKeeper-trunk/1348/])
    ZOOKEEPER-1267. closeSession flag in finalRequestProcessor is superfluous 
(Thomas Koch via phunt)

phunt : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1190450
Files : 
* /zookeeper/trunk/CHANGES.txt
* 
/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/FinalRequestProcessor.java

                
> closeSession flag in finalRequestProcessor is superfluous
> ---------------------------------------------------------
>
>                 Key: ZOOKEEPER-1267
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1267
>             Project: ZooKeeper
>          Issue Type: Sub-task
>            Reporter: Thomas Koch
>            Assignee: Thomas Koch
>            Priority: Trivial
>         Attachments: ZOOKEEPER-1267.patch
>
>
> The variable can be removed and instead where it is evaluated one can just 
> check whether the request.type was OpCode.closesession. Removes one 
> indirection from your head in a method that's long enough already.

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