[jira] [Comment Edited] (ZOOKEEPER-1962) Add a CLI command to recursively list a znode and children

["Abraham Fine (JIRA)" <ji...@apache.org>]

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

Abraham Fine edited comment on ZOOKEEPER-1962 at 8/26/16 6:41 PM:
------------------------------------------------------------------

I agree with [~phunt] that reusing the logic from deleteall (listSubTreeBFS) is likely a good idea

It also may be valuable to add a comment to the code/command output that explains that modifications can occur to the tree while the ls -R command is running and it may be the case that the output does not necessarily represent the actual state of zookeeper at any given point in time.

In other words, the race condition noted above can occur with the creation of nodes, which will not cause any of the listing operations to fail so we would have no way to fail intelligently.


was (Author: abrahamfine):
I agree with [~phunt] that reusing the logic from deleteall (listSubTreeBFS) is likely a good idea

It also may be valuable to add a comment to the code/command output that explains that modifications can occur to the tree while the ls -R command is running and it may be the case that the output does not necessarily represent the actual state of zookeeper at any given point in time.

> Add a CLI command to recursively list a znode and children
> ----------------------------------------------------------
>
>                 Key: ZOOKEEPER-1962
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1962
>             Project: ZooKeeper
>          Issue Type: New Feature
>          Components: java client
>    Affects Versions: 3.4.6
>            Reporter: Gautam Gopalakrishnan
>            Assignee: Gautam Gopalakrishnan
>            Priority: Minor
>             Fix For: 3.5.3, 3.6.0
>
>         Attachments: ZOOKEEPER-1962.diff, ZOOKEEPER-1962_v2.patch, ZOOKEEPER-1962_v3.patch, ZOOKEEPER-1962_v4.patch
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> When troubleshooting applications where znodes can be multiple levels deep  (eg. HBase replication), it is handy to see all child znodes recursively rather than run an ls for each node manually.
> So I propose adding an option to the "ls" command (-r) which will list all child nodes under a given znode. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)