[ https://issues.apache.org/jira/browse/ZOOKEEPER-1272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13139375#comment-13139375 ]
Thomas Koch commented on ZOOKEEPER-1272: ---------------------------------------- Sorry Marshall, what do you want to express? Do you agree, that the following changes should be made: - remove code that sets rc.err in DataTree.processTxn - remove code that conditionally throws an Exception in ZooKeeper.multiInternal() if return value of cnxn.sendRequest != 0 > ZooKeeper.multi() could violate API if server misbehaves > -------------------------------------------------------- > > Key: ZOOKEEPER-1272 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1272 > Project: ZooKeeper > Issue Type: Bug > Reporter: Thomas Koch > Assignee: Thomas Koch > Priority: Minor > > The client API method Zookeeper.multi() promisses, that the KeeperException > it throws in case of one of the multi ops failing, contains a list of > individual results. > The method ZooKeeper.multiInternal() however throws a Keeperexception if the > returned response header has an error code != 0. This should actually never > happen if the server does not misbehave since the error code of a multi > response is always zero, but I managed to trigger this code path with my > refactorings. -- 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