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

Rakesh R commented on HBASE-7847:
---------------------------------

Thanks again [~saint....@gmail.com] for the comments. I'll prepare a new patch 
addressing these. 
{quote} 
Should this be true by default?
+ deleteChildrenRecursivelyMultiOrSequential(zkw, false, node);
i.e. retry with the sequential if the multi fails?
{quote}
Yeah, if true will retry sequentially on multi failures. I'll make this as true.

For the below one, could you help to understand more.
bq. Call out BFS for what it is.

BFS I meant Breadth-First-Search traversal strategy. Whether you are asking to 
avoid abbreviation/short forms ?

> Use zookeeper multi to clear znodes
> -----------------------------------
>
>                 Key: HBASE-7847
>                 URL: https://issues.apache.org/jira/browse/HBASE-7847
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Ted Yu
>            Assignee: Rakesh R
>         Attachments: 7847-v1.txt, 7847_v6.patch, 7847_v6.patch, 
> HBASE-7847.patch, HBASE-7847.patch, HBASE-7847.patch, HBASE-7847_v4.patch, 
> HBASE-7847_v5.patch, HBASE-7847_v6.patch
>
>
> In ZKProcedureUtil, clearChildZNodes() and clearZNodes(String procedureName) 
> should utilize zookeeper multi so that they're atomic



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to