dennishuo commented on code in PR #2672:
URL: https://github.com/apache/polaris/pull/2672#discussion_r2384462221


##########
CHANGELOG.md:
##########
@@ -40,6 +40,8 @@ request adding CHANGELOG notes for breaking (!) changes and 
possibly other secti
 
 - Added a Management API endpoint to reset principal credentials, controlled 
by the `ENABLE_CREDENTIAL_RESET` (default: true) feature flag.
 
+- Added support for S3-compatible storage that does not have STS (use 
`stsUavailable: true` in catalog storage configuration)

Review Comment:
   Yeah, to clarify, I didn't mean to add too much, just maybe adding a `; 
credential vending will then be disabled` in the parentheses so that the change 
note would say
   
       (use `stsUavailable: true` in catalog storage configuration; credential 
vending will then be disabled);
   
   In any case, non-blocking.



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