maoling commented on issue #839: ZOOKEEPER-3290: Throw detailed KeeperException when a transaction failed URL: https://github.com/apache/zookeeper/pull/839#issuecomment-469107877 +1 - don't look at the specific code,just take a test,it will be ok to throw the error path. - also think about backward compatibility,take a quick test to show I'am overreacting,no backward compatibility problem with the change of the protocol:`ErrorTxn`. 1.low version client(without this patch) connect to high version server(with this patch): `org.apache.zookeeper.KeeperException$NoNodeException: KeeperErrorCode = NoNode` 2.high version client connect to lower version server: `org.apache.zookeeper.KeeperException$NoNodeException: KeeperErrorCode = NoNode`
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
