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 2016/12/14 16:43:58 UTC

[jira] [Commented] (CURATOR-364) Persistent Node Constructor Requires a Started ZK Connection

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

Jordan Zimmerman commented on CURATOR-364:
------------------------------------------

It's because the ctor calls client.create() early as an optimization. I agree that this should be fixed.

> Persistent Node Constructor Requires a Started ZK Connection
> ------------------------------------------------------------
>
>                 Key: CURATOR-364
>                 URL: https://issues.apache.org/jira/browse/CURATOR-364
>             Project: Apache Curator
>          Issue Type: Bug
>          Components: Recipes
>    Affects Versions: 2.11.1
>            Reporter: Ilya Biryukov
>
> *Steps to reproduce:*
> 1. Initialise an instance of PersistentNode
> 2. Ensure that CuratorFramework is Not Started (Latent)
> *Expected behaviour:*
> PersistentNode constructor should work without an active CuratorFramework connection to ZK
> *Actual behaviour*:
> An Exception is thrown:
> {code}
> Caused by: java.lang.IllegalStateException: instance must be started before calling this methodDEBUG [2016-12-14 10:25:25,641] akka.event.EventStream: logger log1-Slf4jLogger started
> 	at com.google.common.base.Preconditions.checkState(Preconditions.java:150)
> 	at org.apache.curator.framework.imps.CuratorFrameworkImpl.create(CuratorFrameworkImpl.java:351)
> 	at org.apache.curator.framework.recipes.nodes.PersistentNode.<init>(PersistentNode.java:173)
> {code}
> *Thoughts*
> This behaviour is inconsistent with other Zk recepies based on Closable interface. PersistentNode should not attempt any Zk communication until start() method is called.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)