Benjamin Reed updated ZOOKEEPER-33:

    Status: Open  (was: Patch Available)

-1 I like the patch, but there are just a couple of things that need fixing:

1) The mapping should be between # and ArrayList<ACL> rather than # and ACL. I 
think it further simplifies things and will make serialization/deserialization 
go even faster. convertACLs should return a long for example.

2) When you serialize the # to ACL map and the ACL to # map, you only need to 
serialize once. I would serialize the # to ACL map. When you deserialize you 
can rebuild the ACL to # map at the same time since they have the same 

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