[ZOOKEEPER-839] deleteRecursive does not belong to the other methods

• Type: Sub-task
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 3.3.1
• Fix Version/s: 3.4.0
• Component/s: java client
• Hadoop Flags: Reviewed
• Tags: atomic operations
• Assignee: Mahadev Konar
• Reporter: Patrick Datko
• Created: 11/Aug/10 14:53
• Updated: 23/Nov/11 19:22
• Resolved: 14/Aug/11 16:41

Description

DeleteRecursive has been committed to trunk already as a method to the
zookeeper class. So in the API it has the same level as the atomic operations
create, delete, getData, setData, etc. The user must get the false impression,
that deleteRecursive is also an atomic operation.
It would be better to have deleteRecursive in some helper class but not that
deep in zookeeper's core code. Maybe I'd like to have another policy on how to
react if deleteRecursive fails in the middle of its work?