Support S3 cross region access property #25200
Conversation
|
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla |
1 similar comment
|
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla |
heechan3006
force-pushed
the
feature/aws-sdk
branch
from
9a7d2d0 to
3a17b5b
Compare
|
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla |
|
Rebase the branch |
| @@ -570,6 +571,19 @@ public S3FileSystemConfig setSupportsExclusiveCreate(boolean supportsExclusiveCr | |||
| return this; | |||
| } | |||
|
|
|||
| public boolean getCrossRegionAccessEnabled() | |||
There was a problem hiding this comment.
| return crossRegionAccessEnabled; | ||
| } | ||
|
|
||
| @Config("s3.cross-region-access-enabled") |
There was a problem hiding this comment.
just s3.cross-region-access
There was a problem hiding this comment.
| } | ||
|
|
||
| @Config("s3.cross-region-access-enabled") | ||
| @ConfigDescription("Whether S3 cross region access enabled") |
There was a problem hiding this comment.
Enable S3 cross region access
There was a problem hiding this comment.
heechan3006
force-pushed
the
feature/aws-sdk
branch
from
3a17b5b to
eeff914
Compare
|
Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla |
Description
For AWS Java sdk v2, S3 bucket cross region access is restricted as a default unlike v1. To meet backward compatibility, cross region access property should be supported.
https://trinodb.slack.com/archives/CGB0QHWSW/p1740554905054189
Additional context and related issues
Release notes
( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text: