[
https://issues.apache.org/jira/browse/CURATOR-666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17707985#comment-17707985
]
Zili Chen commented on CURATOR-666:
-----------------------------------
Oops, I confuse {{CreateBuilderImpl}} and {{CuratorEventImpl}}. I agree that
the ctor looks weird now:
{code:java}
this.path = client.unfixForNamespace(path);
this.name = name;
{code}
> Background create strip namespace twice from path
> -------------------------------------------------
>
> Key: CURATOR-666
> URL: https://issues.apache.org/jira/browse/CURATOR-666
> Project: Apache Curator
> Issue Type: Bug
> Components: Framework
> Affects Versions: 5.4.0
> Reporter: Kezhu Wang
> Assignee: Kezhu Wang
> Priority: Major
>
> {{CreateBuilderImpl::sendBackgroundResponse}} strips namespace twice:
> * One in body of {{sendBackgroundResponse}} or newly introduced
> {{createResponseEvent}}.
> * One in construction of {{CuratorEventImpl}}.
> This could cause incorrect resulting path if we are creating "/zoo" under
> "/zoo" namespace.
> This recall me ZOOKEEPER-4601, I verified that
> {{CuratorFramework::getConfig}} get "keeper/config" in "/zoo" and "/config"
> in "/zookeeper" namespace. We might need extra coordination with ZooKeeper
> community to maintain consistent between curator and zookeeper but I think we
> could fix `CreateBuilderImpl` independently. I remembered that I sent email
> in zookeeper dev list before, I will bump that up for discussion.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
