You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2020/04/24 11:11:03 UTC

[GitHub] [hadoop-ozone] captainzmc opened a new pull request #868: HDDS-3373. Fix ozonefs put and mkdir KEY_NOT_FOUND issue when ACL enable

captainzmc opened a new pull request #868:
URL: https://github.com/apache/hadoop-ozone/pull/868


   ## What changes were proposed in this pull request?
   
       When we put file using O3FS, FileSystem will check if it exists first. If the key not exist, checkAccess will report KEY_NOT_FOUND Exception when get keyInfo. 
       Because the previous bucket permission check has passed, it’s currently reading a key that does not exist, so this should be allowed.
       So when KEY_NOT_FOUND appears, we confirm whether it is a read ACLType, and if it’s a read ACLType we return true. To avoid KEY_NOT_FOUND exception.
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-3457
   
   ## How was this patch tested?
   
   Enable acl and put file or mkdir using o3fs.
   


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org


[GitHub] [hadoop-ozone] aryangupta1998 commented on pull request #868: HDDS-3457. Fix ozonefs put and mkdir KEY_NOT_FOUND issue when ACL enable

Posted by GitBox <gi...@apache.org>.
aryangupta1998 commented on pull request #868:
URL: https://github.com/apache/hadoop-ozone/pull/868#issuecomment-684394220


   Hi @captainzmc,
   I enabled ACL and performed mkdir using o3fs, but 'checkAccess' function is never reached. I have put some System.out.println statements in 'checkAccess' function, but nothing is being printed. Can you please help!


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org


[GitHub] [hadoop-ozone] captainzmc commented on pull request #868: HDDS-3457. Fix ozonefs put and mkdir KEY_NOT_FOUND issue when ACL enable

Posted by GitBox <gi...@apache.org>.
captainzmc commented on pull request #868:
URL: https://github.com/apache/hadoop-ozone/pull/868#issuecomment-685244606


   > Hi @captainzmc,
   > I enabled ACL and performed mkdir using o3fs, but 'checkAccess' function is never reached. I have put some System.out.println statements in 'checkAccess' function, but nothing is being printed. Can you please help!
   
   Hi @aryangupta1998 [Is that the method](https://github.com/apache/hadoop-ozone/blob/34ee8311b0d0a37878fe1fd2e5d8c1b91aa8cc8f/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java#L1633)? I suggest you print the log with LOG.xxx or enable debugging in IDEA.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org