You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@curator.apache.org by "Jordan Zimmerman (JIRA)" <ji...@apache.org> on 2013/09/17 17:41:51 UTC
[jira] [Comment Edited] (CURATOR-34)
CuratorFramework.create().creatingParentsIfNeeded().forPath() throws
NodeExistsException if node exists
[ https://issues.apache.org/jira/browse/CURATOR-34?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13769615#comment-13769615 ]
Jordan Zimmerman edited comment on CURATOR-34 at 9/17/13 3:40 PM:
------------------------------------------------------------------
Seems like it would be: {{create.ifNotExists().forPath(...)}}. It would chain with other builder methods so you could do: {{create.creatingParentsIfNeeded().ifNotExists().forPath(...)}}, etc.
Technically, though, as is pointed out above, this can be accomplished with {{EnsurePath}}. Is this API really needed?
was (Author: randgalt):
Seems like it would be: {{create.ifNotExists().forPath(...)}}. It would chain with other builder methods so you could do: {{create.creatingParentsIfNeeded().ifNotExists().forPath(...)}}, etc.
> CuratorFramework.create().creatingParentsIfNeeded().forPath() throws NodeExistsException if node exists
> -------------------------------------------------------------------------------------------------------
>
> Key: CURATOR-34
> URL: https://issues.apache.org/jira/browse/CURATOR-34
> Project: Apache Curator
> Issue Type: Improvement
> Components: Framework
> Affects Versions: 2.0.0-incubating, 2.0.1-incubating
> Reporter: Nandor Kracser
> Priority: Minor
>
> I think that NodeExistsException should be ignored in CreateBuilderImpl. In the try and in the NoNodeException catch block as well.
> org.apache.zookeeper.KeeperException$NodeExistsException: KeeperErrorCode = NodeExists for /examples/something
> at org.apache.zookeeper.KeeperException.create(KeeperException.java:119)
> at org.apache.zookeeper.KeeperException.create(KeeperException.java:51)
> at org.apache.zookeeper.ZooKeeper.create(ZooKeeper.java:783)
> at org.apache.curator.framework.imps.CreateBuilderImpl$10.call(CreateBuilderImpl.java:626)
> at org.apache.curator.framework.imps.CreateBuilderImpl$10.call(CreateBuilderImpl.java:610)
> at org.apache.curator.RetryLoop.callWithRetry(RetryLoop.java:107)
> at org.apache.curator.framework.imps.CreateBuilderImpl.pathInForeground(CreateBuilderImpl.java:606)
> at org.apache.curator.framework.imps.CreateBuilderImpl.forPath(CreateBuilderImpl.java:429)
> at org.apache.curator.framework.imps.CreateBuilderImpl.forPath(CreateBuilderImpl.java:409)
> at org.apache.curator.framework.imps.CreateBuilderImpl$4.forPath(CreateBuilderImpl.java:317)
> at org.apache.curator.framework.imps.CreateBuilderImpl$4.forPath(CreateBuilderImpl.java:253)
--
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