You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@curator.apache.org by "Scott Blum (JIRA)" <ji...@apache.org> on 2014/07/29 02:41:39 UTC
[jira] [Created] (CURATOR-128) There is no namespace-safe way to
refer to the root node
Scott Blum created CURATOR-128:
----------------------------------
Summary: There is no namespace-safe way to refer to the root node
Key: CURATOR-128
URL: https://issues.apache.org/jira/browse/CURATOR-128
Project: Apache Curator
Issue Type: Bug
Components: Framework
Reporter: Scott Blum
Add the following test to TestNamespaceFacade:
{code}
@Test
public void testRootAccess() throws Exception
{
CuratorFramework client = CuratorFrameworkFactory.newClient(server.getConnectString(), new RetryOneTime(1));
try
{
client.start();
client.create().forPath("/one");
Assert.assertNotNull(client.getZookeeperClient().getZooKeeper().exists("/one", false));
Assert.assertNotNull(client.checkExists().forPath("/"));
try
{
client.checkExists().forPath("");
Assert.fail("IllegalArgumentException expected");
}
catch ( IllegalArgumentException expected )
{
}
Assert.assertNotNull(client.usingNamespace("one").checkExists().forPath(""));
try
{
client.usingNamespace("one").checkExists().forPath("/");
Assert.fail("IllegalArgumentException expected");
}
catch ( IllegalArgumentException expected )
{
}
}
finally
{
CloseableUtils.closeQuietly(client);
}
}
{code}
This tests PASSES, which means that there's no canonical way to refer to the root node. If the client is not namespaced, "/" works and "" does not work. If the client is namespaced, "" works and "/" does not.
In either case, I think ZKPaths.makePath mishandles certain cases.
If you append "/foo" and "/" the result is "/foo/" which is an invalid path.
On the other hand, if you append "" and "bar" the result is "//bar" which is also invalid.
What's the right behavior here? Does the root node / root of a namespace always need to be referred to as "/" or is empty string an acceptable alias?
--
This message was sent by Atlassian JIRA
(v6.2#6252)