bharatviswa504 commented on a change in pull request #1104:
URL: https://github.com/apache/hadoop-ozone/pull/1104#discussion_r447301716
##########
File path:
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java
##########
@@ -2145,37 +2168,51 @@ public OmKeyLocationInfo allocateBlock(OmKeyArgs args,
long clientID,
*/
@Override
public OmKeyInfo lookupKey(OmKeyArgs args) throws IOException {
+ ResolvedBucket bucket = resolveBucketLink(args);
Review comment:
Now for each operation if it is link bucket where if it has
sourceVolume/SourceBucket we do checkAcls twice. One with READ permission on
sourceBucket/SourceVolume in resoleBucketLink and one in actual request with
required ACL type. It is not clear why do we need the first check acl.
If it requires, do you think we need to have an API where we can check all
required ACLS with a single checkAcl call?
----------------------------------------------------------------
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org
