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

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

Define PATH_SEPARATOR
---------------------

                 Key: ZOOKEEPER-277
                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-277
             Project: Zookeeper
          Issue Type: Improvement
            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.


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

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-277?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-277:
------------------------------------

    Fix Version/s: 3.3.0

> 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
>             Fix For: 3.3.0
>
>
> 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.


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

Posted by "Nitay Joffe (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ZOOKEEPER-277?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12671565#action_12671565 ] 

Nitay Joffe commented on ZOOKEEPER-277:
---------------------------------------

Should also add some constant, say ZNODE_ANY_VERSION, for accepting any version, that is -1.

> 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.


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

Posted by "Patrick Hunt (JIRA)" <ji...@apache.org>.
    [ 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.


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

Posted by "Patrick Hunt (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-277?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Hunt updated ZOOKEEPER-277:
-----------------------------------

    Component/s: tests
                 server
                 java client
                 documentation
                 c client

> 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.