[GitHub] [ozone] sodonnel commented on pull request #3103: HDDS-6321. Avoid refresh pipeline for key lookup in checkAcls

[GitBox <gi...@apache.org>]

sodonnel commented on pull request #3103:
URL: https://github.com/apache/ozone/pull/3103#issuecomment-1048940846


   If we don't refresh the locations, does the returned KeyInfo still have locations that were stored in OM, but they may be stale? I guess the change here is for checkAccess, where `.setHeadOp(true)` is set to true now, and it is more of an internal call, so it probably doesn't matter.
   
   How does this change impact TrashOzoneFileSystem and OzonePrefixPathImpl as `setHeadOp` is set to true there too?. I guess I am not too familiar with the OM code, so I am just trying to understand how this change fits into improving the ACL checks.


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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