Benjamin Reed updated ZOOKEEPER-33:

    Status: Open  (was: Patch Available)

-1 Sorry Mahadev, two small issues (one is super trivial :)

1) convertLongs should be called covertLong since there is only a single Long 
being converted
2) convertAcls should use a Map<List<ACL>, Long> to convert an ACL list to a 
long, otherwise that could become an expensive method when more than a few 
different ACLs are used. Note, that Map does not need to be serialized and can 
be deserialized with the Map<Long,List<ACL>>

> Better ACL management
> ---------------------
>                 Key: ZOOKEEPER-33
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-33
>             Project: Zookeeper
>          Issue Type: New Feature
>          Components: server
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>             Fix For: 3.0.0
>         Attachments: ZOOKEEPER-33.patch, ZOOKEEPER-33.patch, 
> ZOOKEEPER-33.patch, ZOOKEEPER-33.patch, ZOOKEEPER-33.patch, ZOOKEEPER-33.patch
> Moved from SourceForge to Apache.
> http://sourceforge.net/tracker/index.php?func=detail&aid=1945175&group_id=209147&atid=1008547

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