[GitHub] TisonKun opened a new pull request #839: ZOOKEEPER-3290: Throw detailed KeeperException when a transaction failed

[GitBox <gi...@apache.org>]

TisonKun opened a new pull request #839: ZOOKEEPER-3290: Throw detailed KeeperException when a transaction failed
URL: https://github.com/apache/zookeeper/pull/839
 
 
   Assume we execute the follow statements
   
   ```java
   ZooKeeper zk = ...;
   zk.multi(Arrays.asList(
     Op.check(path1, -1),
     Op.delete(path2, -1)));
   ```
   
   If path1 or path2 didn't exist, we got an exception KeeperException.NoNodeException without which of them doesn't exist.
   
   The reason is when we executed PrepRequestProccessor#pRequest in PrepRequestProccessor#L804, it processed KeeperException.NoNodeException which contained path info.
   
   However, we generated ErrorTxn which only contains err field represented error code and lost path info. I try a resolution that extend ErrorTxn to contain path info.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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