-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[[fix](rest-s3)Set AWS Request Checksum Calculation only if not set void issues on non-S3 storage #58467
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
…void issues on non-S3 storage AWS SDK for Java v2 uses the system property AwsSystemSetting.AWS_REQUEST_CHECKSUM_CALCULATION to control request payload checksum calculation. According to the official discussion AWS SDK v2 Discussion apache#5802 : By default, the SDK may automatically calculate checksums when required. For AWS S3, setting the default value "WHEN_REQUIRED" ensures checksums are calculated only when necessary, providing both compatibility and performance benefits. aws-chunked encoding is not supported with the specified x-amz-content-sha256 value because these services do not support AWS-specific chunked encoding or SHA256 checksum.
d5e6e1e to
5251cb7
Compare
|
run buildall |
FE UT Coverage ReportIncrement line coverage |
TPC-H: Total hot run time: 35039 ms |
TPC-DS: Total hot run time: 184661 ms |
ClickBench: Total hot run time: 27.31 s |
FE Regression Coverage ReportIncrement line coverage |
morningman
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
…oid issues on non-S3 storage (#58467) AWS SDK for Java v2 uses the system property AwsSystemSetting.AWS_REQUEST_CHECKSUM_CALCULATION to control request payload checksum calculation. According to the official discussion AWS SDK v2 Discussion aws/aws-sdk-java-v2#5802 By default, the SDK may automatically calculate checksums when required. For AWS S3, setting the default value "WHEN_REQUIRED" ensures checksums are calculated only when necessary, providing both compatibility and performance benefits. For non-S3 object storage (such as COS, OSS, OBS), using this default value may cause errors, for example: `aws-chunked encoding is not supported with the specified x-amz-content-sha256 value` because these services do not support AWS-specific chunked encoding or SHA256 checksum.
…f not set void issues on non-S3 storage (apache#58467) apache#58467
AWS SDK for Java v2 uses the system property AwsSystemSetting.AWS_REQUEST_CHECKSUM_CALCULATION to control request payload checksum calculation.
According to the official discussion AWS SDK v2 Discussion aws/aws-sdk-java-v2#5802
By default, the SDK may automatically calculate checksums when required.
For AWS S3, setting the default value "WHEN_REQUIRED" ensures checksums are calculated only when necessary, providing both compatibility and performance benefits.
For non-S3 object storage (such as COS, OSS, OBS), using this default value may cause errors, for example:
aws-chunked encoding is not supported with the specified x-amz-content-sha256 valuebecause these services do not support AWS-specific chunked encoding or SHA256 checksum.