ChenSammi commented on code in PR #7455:
URL: https://github.com/apache/ozone/pull/7455#discussion_r1865252806
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/key/OMKeyRequest.java:
##########
@@ -452,12 +469,15 @@ protected List< OzoneAcl > getAclsForKey(KeyArgs keyArgs,
* @param keyArgs
* @param bucketInfo
* @param omPathInfo
+ * @param config
* @return Acls which inherited parent DEFAULT and keyArgs ACCESS acls.
*/
- protected static List<OzoneAcl> getAclsForDir(KeyArgs keyArgs,
- OmBucketInfo bucketInfo, OMFileRequest.OMPathInfo omPathInfo) {
+ protected List<OzoneAcl> getAclsForDir(KeyArgs keyArgs, OmBucketInfo
bucketInfo,
+ OMFileRequest.OMPathInfo omPathInfo, OzoneConfiguration config) throws
OMException {
// Acls inherited from parent or bucket will convert to DEFAULT scope
List<OzoneAcl> acls = new ArrayList<>();
+ // add default ACLs
+ acls.addAll(getDefaultAclList(createUGIForApi(), config));
Review Comment:
De-duplication is handled by L495.
--
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]
