andrew4699 opened a new pull request, #437:
URL: https://github.com/apache/polaris/pull/437

   # Description
   
   In a previous change I took a stab at transforming some 
cloud-provider-specific 403s into Polaris 403s. I don't think the hints were 
fully accurate as I'm seeing AWS errors that don't mention "access denied". 
Rather than trying to chase down the exact error messages per cloud provider, I 
went with reasonable catch-alls. We still only apply this to 
cloud-provider-specific exception types such as S3Exception so this should be 
fine.
   
   ## Type of change
   
   - [X] Bug fix (non-breaking change which fixes an issue)
   
   # How Has This Been Tested?
   
   - [X] Existing unit tests
   
   # Checklist:
   
   - [X] I have performed a self-review of my code
   - [X I have commented my code, particularly in hard-to-understand areas
   - [ ] I have made corresponding changes to the documentation
   - [X] My changes generate no new warnings
   - [ ] If adding new functionality, I have discussed my implementation with 
the community using the linked GitHub issue
   


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