[ZOOKEEPER-3290] Throw detailed KeeperException when a transaction failed

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 3.5.4, 3.4.13
• Component/s: server
• Labels: pull-request-available
• Assignee: Zili Chen
• Reporter: Zili Chen
• Created: 26/Feb/19 08:10
• Updated: 11/May/23 19:03
• Resolved: 11/May/23 19:03

Description

Assume we execute the follow statements

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. Maybe a reasonable resolution is extend ErrorTxn to contain path info or a general data byte array.

Issue Links

• duplicates: ZOOKEEPER-2276 Multi operation failure does not include path in KeeperException
• relates to: ZOOKEEPER-3289 Throw KeeperException with path in DataTree operations
• links to: GitHub Pull Request #839