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

Benjamin Reed commented on ZOOKEEPER-849:
-----------------------------------------

-1 unfortunately, we need to change the direction of this change a bit:

1. on the server, we want to start using byte[], we don't want to go to Path 
class. we definitely do not want to create a Path object to do object 
validation, so we need to keep the utility method in PathUtils

2. the isSequential() flag seems very problematic. you aren't going to be able 
to figure out that flag from any path returned from the server, so we really 
shouldn't have it. i'm not clear where this could be used since the sequential 
flag is a create parameter.

3. i'm not sure what the motivation for the Path class is. can you motivate a 
bit more.

btw, ZOOKEEPER-12 refers to tacking the parent directory prefix on returned 
strings, not to creating a new class.

> Provide Path class
> ------------------
>
>                 Key: ZOOKEEPER-849
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-849
>             Project: ZooKeeper
>          Issue Type: Sub-task
>          Components: java client
>            Reporter: Thomas Koch
>            Assignee: Thomas Koch
>             Fix For: 3.4.0
>
>         Attachments: ZOOKEEPER-849.patch, ZOOKEEPER-849.patch
>
>


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

Reply via email to