lefebsy commented on code in PR #389:
URL: https://github.com/apache/polaris/pull/389#discussion_r1829782885
##########
spec/polaris-management-service.yml:
##########
@@ -901,6 +903,58 @@ components:
required:
- roleArn
+ S3StorageConfigInfo:
Review Comment:
I think it will not be too hard to unify, in a next step. I miss AWS access
to do tests, but for what I know or seen :
- STS endpoint is a AWS IAM endpoint, in S3 compatible solutions, when it is
available, it is merged with S3 endpoint.
- Region is available in S3 compatible solutions, but not used a lot, or
mostly implemented to be compliant with aws sdk clients.
- about the cross region, it is more related to Polaris and Iceberg with
the Iceberg overloaded S3fileIO, but I seen that
https://github.com/apache/iceberg/pull/11259 have been merged 3 weeks ago -->
so next Iceberg version will be ok soon.
MinIO claim that their product API is 100% compatible with AWS S3 API.
Almost the same for many alternatives...
- The S3 Compatible implementation could easily propose an optional
parameter "arnRole" like the mandatory one in the existing aws class, with less
regexp patern to allow more flexibility for some implementation where "aws"
inside the string is replaced by the product name (exemple "ecs" for DELL
ECS)... It could help for a smooth transition
--
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]
