AWS: Include http-auth-aws-crt module into iceberg-aws-bundle#10972
Conversation
There was a problem hiding this comment.
LGTM, thanks @aajisaka !
Looks like this got broken as part of this re-factor, #8261 and we didn't run integ tests as part of this (it's presently manual). can you please also run integ test to ensure this works e2e as well, we have an integ test for this here : https://github.com/apache/iceberg/blob/main/aws/src/integration/java/org/apache/iceberg/aws/s3/TestS3FileIOIntegration.java#L170
|
Thank you @singhpk234. There're existing tests for cross-region access point, but there's no test for multi-region access point. Added integration tests for multi-region access point and ran them successfully.
|
singhpk234
left a comment
There was a problem hiding this comment.
LGTM, Thanks @aajisaka !
cc @amogh-jahagirdar @nastra for review and merge
|
Ran the new integration tests without adding the dependency: testNewInputStreamWithMultiRegionAccessPoint() testNewOutputStreamWithMultiRegionAccessPoint() |
amogh-jahagirdar
left a comment
There was a problem hiding this comment.
LGTM, thanks @aajisaka! thanks for reviewing @singhpk234 @nastra! I'll merge when the checks pass
|
Thank you @singhpk234 @nastra and @amogh-jahagirdar ! |
Include
software.amazon.awssdk:http-auth-aws-crtinto iceberg-aws-bundle to enable access to S3 multi-region access points.Closes #10967