CLDSRV-954: use native crc32 and crc32c#6226
Conversation
Hello leif-scality,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the |
❌ 1 Tests Failed:
View the full list of 1 ❄️ flaky test(s)
To view more test analytics, go to the Test Analytics Dashboard |
3a9f4f7 to
7f2629b
Compare
| // Native CRC32/CRC32C via the AWS Common Runtime (aws-crt): hardware-accelerated | ||
| // and dramatically faster than the pure-JS @aws-crypto/crc32{,c}. aws-crt is | ||
| // already loaded in-process by @aws-sdk/crc64-nvme-crt below. |
There was a problem hiding this comment.
Comment could be shorter
| // Native CRC32/CRC32C via the AWS Common Runtime (aws-crt): hardware-accelerated | |
| // and dramatically faster than the pure-JS @aws-crypto/crc32{,c}. aws-crt is | |
| // already loaded in-process by @aws-sdk/crc64-nvme-crt below. | |
| // Native hardware-accelerated, faster than pure-JS @aws-crypto/crc32{,c} |
7f2629b to
9d598b5
Compare
|
/approve |
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the The following options are set: approve |
|
ping |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
This pull request did not target the following hotfix branch(es) so they
Please check the status of the associated issue CLDSRV-954. Goodbye leif-scality. The following options are set: approve |
No description provided.