[ 
https://issues.apache.org/jira/browse/CURATOR-30?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13701002#comment-13701002
 ] 

Jordan Zimmerman commented on CURATOR-30:
-----------------------------------------

Looks good. A few things though:

* deletingChildrenIfNeeded() should also support guaranteed(). This makes 
writing the fluent ifaces a lot more work. deletingChildrenIfNeeded() will need 
to be moved into a separate interface, etc.

* The static deleteChildren() method uses the raw ZooKeeper handle. Instead it 
should use a CuratorFramework handle so that retries, etc. are used.

                
> Recursive delete
> ----------------
>
>                 Key: CURATOR-30
>                 URL: https://issues.apache.org/jira/browse/CURATOR-30
>             Project: Apache Curator
>          Issue Type: Improvement
>          Components: Client, Framework
>            Reporter: John Vines
>         Attachments: CURATOR-30.patch
>
>
> Currently there is the ability to recursive create parent znodes when you 
> create a node. However, there is no ability to recursively delete a 
> hierarchy. Zookeeper already provides this in their ZKUtil.java package, but 
> it seems like a very curator-ish thing to perform as well. There is the 
> potential difficulty involved with the guarantee() functionality, but it 
> should be workable.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to