You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Patrick Hunt (JIRA)" <ji...@apache.org> on 2009/01/24 01:53:59 UTC

[jira] Commented: (ZOOKEEPER-277) Define PATH_SEPARATOR

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-277?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12666813#action_12666813 ] 

Patrick Hunt commented on ZOOKEEPER-277:
----------------------------------------

+1 - sounds fine to me.

Note for implementor: name the constant something like ZNODE_PATH_SEPARATOR to ensure that ppl understand that this is znode sep (although that's pretty long).

Also, if this is done it should be done for both java as well as c client binding. Update docs and tests.


> Define PATH_SEPARATOR
> ---------------------
>
>                 Key: ZOOKEEPER-277
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-277
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: c client, documentation, java client, server, tests
>            Reporter: Nitay Joffe
>            Priority: Trivial
>
> We should define a constant for PATH_SEPARATOR = "/" and use that throughout the code rather than the hardcoded "/". Users can be told to use this constant to be safe in case of future changes.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.