zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [zookeeper] maoling commented on issue #839: ZOOKEEPER-3290: Throw detailed KeeperException when a transaction failed
Date Mon, 04 Mar 2019 03:39:39 GMT
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

Mime
View raw message