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

Jeffrey Zhong commented on HBASE-8509:
--------------------------------------

[~enis] A good point.
The function isn't thread safe regarding parent folder creation so the issue 
you mentioned is expected behavior. Before, it might cause a ugly situation if 
other threads retry deletion so deletion & creation will step on each others's 
toe for a while

I can simulate old behavior to handle NoNodeException again but it's better 
with current patch IMHO which is more clean. 
                
> ZKUtil#createWithParents won't set data during znode creation when parent 
> folder doesn't exit
> ---------------------------------------------------------------------------------------------
>
>                 Key: HBASE-8509
>                 URL: https://issues.apache.org/jira/browse/HBASE-8509
>             Project: HBase
>          Issue Type: Bug
>          Components: Zookeeper
>            Reporter: Jeffrey Zhong
>            Assignee: Jeffrey Zhong
>             Fix For: 0.98.0, 0.94.8, 0.95.1
>
>         Attachments: hbase-8509.patch
>
>
> As you can see below, we don't pass data down when parent folder doesn't 
> exists.
> {code}
>     } catch(KeeperException.NoNodeException nne) {
>       createWithParents(zkw, getParent(znode));
>       createWithParents(zkw, znode);
> {code}

--
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