ChenSammi commented on code in PR #4559: URL: https://github.com/apache/ozone/pull/4559#discussion_r1174976184
########## hadoop-hdds/docs/content/design/volume-management.md: ########## @@ -125,11 +125,12 @@ Note: Sanjay is added to the authors as the original proposal of this approach. * `bucket link` operation creates a link bucket. Links are like regular buckets, stored in DB the same way, but with two new, optional pieces of information: source volume and bucket. (The bucket being referenced by the link is called "source", not "target", to follow symlink terminology.) * Link buckets share the namespace with regular buckets. If a bucket or link with the same name already exists, a `BUCKET_ALREADY_EXISTS` result is returned. * Link buckets are not inherently specific to a user, access is restricted only by ACL. - * Link buckets themselves do not store ACLs. All ACLs operations (eg. addacl/getacl/setacl/removeacl) and checks on the link bucket will be applied to the source bucket. + * Link buckets retain their owner ACLs, which are inherited from the default ACLs of their volume. + * All add/set/remove ACL operation proxy to the source bucket. Getacl operation of a link bucket shows source bucket by default (Use option to display ACLs of the link bucket itself). Review Comment: "Getacl operation of a link bucket shows source bucket by default (Use option to display ACLs of the link bucket itself)." Here it discusses the behavior of APIs, instead of CLI. So I guess we should change it to "Getacl operation of a link bucket shows link bucket's ACL". -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
