[jira] Updated: (ZOOKEEPER-444) perms definition for PERMS_ALL differ in C and java

2009-06-17 Thread Mahadev konar (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-444?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mahadev konar updated ZOOKEEPER-444:


  Resolution: Fixed
Release Note: Changed PERMS.ALL definition in ZooDefs in java to include 
ADMIN. This changes the definition of OPEN_ACL_UNSAFE to have admin perms.
Hadoop Flags: [Incompatible change, Reviewed]
  Status: Resolved  (was: Patch Available)

I just committed this. thanks ben for the brilliant review :).

 perms definition for PERMS_ALL differ in C and java
 ---

 Key: ZOOKEEPER-444
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-444
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.1.1
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.0

 Attachments: ZOOKEEPER-444.patch


 the perms_all definition in Java is PERMS.ALL and does not include ADMIN 
 perms but in c the PERMS_ALL def includes the ADMIN perms. We should make it 
 consistent to include or not include the admin perms in both c and java.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-444) perms definition for PERMS_ALL differ in C and java

2009-06-16 Thread Mahadev konar (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-444?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mahadev konar updated ZOOKEEPER-444:


Status: Patch Available  (was: Open)

 perms definition for PERMS_ALL differ in C and java
 ---

 Key: ZOOKEEPER-444
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-444
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.1.1
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.0

 Attachments: ZOOKEEPER-444.patch


 the perms_all definition in Java is PERMS.ALL and does not include ADMIN 
 perms but in c the PERMS_ALL def includes the ADMIN perms. We should make it 
 consistent to include or not include the admin perms in both c and java.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (ZOOKEEPER-444) perms definition for PERMS_ALL differ in C and java

2009-06-16 Thread Mahadev konar (JIRA)

 [ 
https://issues.apache.org/jira/browse/ZOOKEEPER-444?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mahadev konar updated ZOOKEEPER-444:


Attachment: ZOOKEEPER-444.patch

this patch adds admin to ALL in java.

 perms definition for PERMS_ALL differ in C and java
 ---

 Key: ZOOKEEPER-444
 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-444
 Project: Zookeeper
  Issue Type: Bug
Affects Versions: 3.1.1
Reporter: Mahadev konar
Assignee: Mahadev konar
Priority: Blocker
 Fix For: 3.2.0

 Attachments: ZOOKEEPER-444.patch


 the perms_all definition in Java is PERMS.ALL and does not include ADMIN 
 perms but in c the PERMS_ALL def includes the ADMIN perms. We should make it 
 consistent to include or not include the admin perms in both c and java.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.