slfan1989 opened a new issue, #4159:
URL: https://github.com/apache/amoro/issues/4159

   ### Search before asking
   
   - [x] I have searched in the 
[issues](https://github.com/apache/amoro/issues?q=is%3Aissue) and found no 
similar issues.
   
   
   ### What would you like to be improved?
   
   ### What happened?
   In `MixedTableOperations.java`, the `metadataFileLocation()` and 
`locationProvider()` methods are missing the `authenticatedFileIO.doAs()` 
wrapper, while the same methods in the `temp()` implementation do have this 
wrapper. This inconsistency can lead to permission errors in authenticated 
environments (e.g., Kerberos).
   
   #### Expected behavior: 
   All file I/O operations should be wrapped with `authenticatedFileIO.doAs()` 
to ensure consistent authentication context.
   
   #### Actual behavior: 
   The `metadataFileLocation()` and `locationProvider()` methods in the main 
class lack the `doAs()` wrapper, while other methods like `current()`, 
`refresh()`, and `commit()` properly use it.
   
   ### How should we improve?
   
   add the missing `doAs()` wrappers to ensure all operations run with the 
correct authentication context.
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Subtasks
   
   _No response_
   
   ### Code of Conduct
   
   - [x] I agree to follow this project's [Code of 
Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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]

Reply via email to