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

Posted to dev@zookeeper.apache.org by GitBox <gi...@apache.org> on 2019/03/04 03:39:39 UTC

[GitHub] [zookeeper] maoling commented on issue #839: ZOOKEEPER-3290: Throw detailed KeeperException when a transaction failed

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:
users@infra.apache.org


With regards,
Apache Git Services