Skip to content

Comments

Merge additional checksum algorithms support#6744

Merged
zoewangg merged 5 commits intomasterfrom
feature/master/addMoreChecksumAlgos
Feb 20, 2026
Merged

Merge additional checksum algorithms support#6744
zoewangg merged 5 commits intomasterfrom
feature/master/addMoreChecksumAlgos

Conversation

@zoewangg
Copy link
Contributor

Motivation and Context

Add support for additional checksum algorithms: XXHASH64, XXHASH3, XXHASH128, SHA512.

  • The XXHASH algorithms require the AWS CRT library.
  • MD5 is not supported for flexible checksums (httpChecksum trait). MD5 can only be used when a pre-calculated MD5 header is provided. Attempting to specify MD5 as the checksum algorithm throws IllegalArgumentException.
  • MD5 response validation is skipped in HttpChecksumUtils.shouldSkipAlgorithm() since MD5 is not a valid flexible checksum algorithm

Note that the main change has been reviewed an approved in #6742. This PR just merges the feature branch to mainline, adds changelog entry, and bumps minor version.

@zoewangg zoewangg requested a review from a team as a code owner February 20, 2026 03:43
@zoewangg zoewangg added the api-surface-area-approved-by-team Indicate API surface area introduced by this PR has been approved by team label Feb 20, 2026
@zoewangg zoewangg force-pushed the feature/master/addMoreChecksumAlgos branch from edacf01 to b098a7f Compare February 20, 2026 17:40
* Add additional checksum algorithms

* Update checksum order based on perf test and update benchmark test code

* Address feedback

* Fix codegen tests

* Fix codegen generated classes tests
@zoewangg zoewangg force-pushed the feature/master/addMoreChecksumAlgos branch from 25b497c to 25511ca Compare February 20, 2026 21:08
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
66.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@zoewangg zoewangg enabled auto-merge February 20, 2026 21:55
@zoewangg zoewangg added this pull request to the merge queue Feb 20, 2026
Merged via the queue into master with commit 3583c7f Feb 20, 2026
37 of 40 checks passed
@github-actions
Copy link

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api-surface-area-approved-by-team Indicate API surface area introduced by this PR has been approved by team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants