You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Patrick Hunt (JIRA)" <ji...@apache.org> on 2008/06/11 00:35:45 UTC
[jira] Created: (ZOOKEEPER-33) Better ACL management
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
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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Attachment: ZOOKEEPER-33.patch
forgot to add the test...
> 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.
[jira] Work started: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Work on ZOOKEEPER-33 started by Mahadev konar.
> 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
>
>
> 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.
[jira] Work stopped: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Work on ZOOKEEPER-33 stopped by Mahadev konar.
> 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
>
>
> 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Attachment: ZOOKEEPER-33.patch
changed the comment... thanks for the review ben...
> 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, 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Hadoop Flags: [Incompatible change, Reviewed] (was: [Reviewed, Incompatible change])
Status: Patch Available (was: Open)
> 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, 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.
[jira] Commented: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12639279#action_12639279 ]
Mahadev konar commented on ZOOKEEPER-33:
----------------------------------------
the patch uploaded at 06:59 PM is the latest now :) ...
> 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Status: Patch Available (was: Open)
> 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
>
>
> 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Attachment: ZOOKEEPER-33.patch
this is a patch that includes the tests for this jira. The jira includes
1) create a map from long to acl and acl to long in the datatree
2) serialize the maps to the snapshots
3) change the datanodes to now have a reference to the acl maps ( storing longs instead of the acls)
4) the tests check to see if the acl map works correctly -- storing only a single copy of equal acls and also tests to see if the acl map works after a server is shutdown adn then brought up back again...
> 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
>
>
> 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Attachment: ZOOKEEPER-33.patch
fixed a bug in the old patch :)... the old patch overrides acls after deserializing the acls from the datatrree. I just reassigned the index number of acls maps to the deserialized maximum long from the map.
> 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
>
>
> 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
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 information.
> 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Resolution: Fixed
Hadoop Flags: [Incompatible change, Reviewed] (was: [Reviewed, Incompatible change])
Status: Resolved (was: Patch Available)
I just committed this.
> 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, 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Hadoop Flags: [Incompatible change]
Status: Patch Available (was: Open)
> 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
>
>
> 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Attachment: ZOOKEEPER-33.patch
attaching a new patch. The last patch went stale with checking in of ZOOKEEPER-8.
> 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.
[jira] Commented: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12639277#action_12639277 ]
Mahadev konar commented on ZOOKEEPER-33:
----------------------------------------
just to make sure the patch uploaded at 5:53 pm is the latest one... the jira does not show it as the last patch...
> 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
>
>
> 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.
[jira] Commented: (ZOOKEEPER-33) Better ACL management
Posted by "Hudson (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12639802#action_12639802 ]
Hudson commented on ZOOKEEPER-33:
---------------------------------
Integrated in ZooKeeper-trunk #114 (See [http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/114/])
. Better ACL management (Mahadev Konar)
> 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, 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Attachment: ZOOKEEPER-33.patch
attaching the patch with ben's comments ....
1) fixed
2) fixed
> 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, 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Attachment: ZOOKEEPER-33.patch
here is a patch for this issue. its a preliminray patch. I will be adding tests.
The patch does the following:
has a mapping from long to acls in the datatree
each datanode has a reference to acls.
this only impacts the serialization and deserialization of acls on the datatree.
> 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
>
>
> 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Benjamin Reed updated ZOOKEEPER-33:
-----------------------------------
Hadoop Flags: [Incompatible change, Reviewed] (was: [Incompatible change])
+1 Great job! Thanx Mahadev. Please fix one tiny thing before you commit (you don't need to do another patch): the comment of aclIndex should be "the number of acls that we have in the datatree" rather than "the max number of acls". Right? The latter sounds like an upper bound to the number of acls you can have.
(I would have committed the patch, but you didn't submit it :)
> 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, 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.
[jira] Updated: (ZOOKEEPER-33) Better ACL management
Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ZOOKEEPER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mahadev konar updated ZOOKEEPER-33:
-----------------------------------
Attachment: ZOOKEEPER-33.patch
this pathc addesses the issues pouinted out by ben..
ben can you take a look again?
> 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
>
>
> 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.