[ZOOKEEPER-3289] Throw KeeperException with path in DataTree operations

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Not A Problem
• Affects Version/s: 3.5.4, 3.4.13
• Component/s: server
• Assignee: Unassigned
• Reporter: Zili Chen
• Created: 25/Feb/19 05:19
• Updated: 02/Apr/19 10:35
• Resolved: 26/Feb/19 08:12

Description

Currently, if ZooKeeper delete a znode that does not exist. It throws a KeeperException.NoNodeException without path message. It causes difficulty when user debug with ZooKeeper. For example,

Assume we try to do a transaction(with Curator encapsulation)

client.inTransaction()
   .check().forPath(path1).and()
   .delete().forPath(path2).and()
   .commit()

if the statement throw an exception KeeperException.NoNodeException without path information, we can hardly know that it failed at check or delete.

Thus I propose throws KeeperException with path in DataTree operations. We can achieve this without burden.

Issue Links

• is related to: ZOOKEEPER-3290 Throw detailed KeeperException when a transaction failed