xiaoyuyao commented on a change in pull request #847: HDDS-1539. Implement
addAcl,removeAcl,setAcl,getAcl for Volume. Contributed Ajay Kumar.
URL: https://github.com/apache/hadoop/pull/847#discussion_r288181045
##########
File path: hadoop-ozone/common/src/main/proto/OzoneManagerProtocol.proto
##########
@@ -471,12 +500,55 @@ message OzoneAclInfo {
repeated OzoneAclRights rights = 3;
}
+message GetAclRequest {
+ required OzoneObj obj = 1;
+}
+
+message GetAclResponse {
+ repeated OzoneAclInfo acls = 1;
+}
+
+message AddAclRequest {
+ required OzoneObj obj = 1;
+ required OzoneAclInfo acl = 2;
+}
+
+message AddAclResponse {
+ required bool response = 1;
+}
+
+message RemoveAclRequest {
+ required OzoneObj obj = 1;
+ required OzoneAclInfo acl = 2;
+}
+
+message RemoveAclResponse {
+ required bool response = 1;
+}
+
+message SetAclRequest {
+ required OzoneObj obj = 1;
+ repeated OzoneAclInfo acl = 2;
+}
+
+message SetAclResponse {
+ required bool response = 1;
+}
+
+message DeleteAclRequest {
Review comment:
What's the difference between RemoveAclRequest and DeleteAclRequest? I don't
see the later being used.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
