Github user maoling commented on the issue:
https://github.com/apache/zookeeper/pull/686
- agree with this
> Would be better to use key.startsWith(path) instead of contains to make
sure the path starts with the parent's.
- `getAllChildNumber` is just foreach the treemap,no limit may be also ok.
but `setAcl ` does a recursion which really needs a limit.we can create a
seperate jira to do this.
> Have you considered limiting the access to this command? I'm not sure if
it's needed (say we've merged the recursive flag of setAcl command recently
which didn't have limits either)
BTW.
- this patch needs a unit test.
- Do we need a CLI for this feature?we can create a seperate jira to do
this.
---
