moomindani commented on PR #17177: URL: https://github.com/apache/iceberg/pull/17177#issuecomment-5211045309
Not stale — this has had no review yet. CI is green (40/40). Adds two S3FileIO properties for the checksum behavior introduced in AWS SDK 2.30.0 (`s3.request-checksum-calculation`, `s3.response-checksum-validation`), wired through all four client factories. Validated against MinIO and real S3, as described in the PR body. Context: this is the fix for #14439, where the SDK's default `when_supported` breaks third-party S3-compatible stores that reject checksum headers. The commit from the original reporter is preserved. @nastra @singhpk234 @amogh-jahagirdar you've been handling recent AWS changes — would any of you have time for a review? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
