Skip to content

feat(storage): migrate sync client to unified checksum options#16264

Open
v-pratap wants to merge 5 commits into
googleapis:mainfrom
v-pratap:unified-checksum-options-pr3
Open

feat(storage): migrate sync client to unified checksum options#16264
v-pratap wants to merge 5 commits into
googleapis:mainfrom
v-pratap:unified-checksum-options-pr3

Conversation

@v-pratap

Copy link
Copy Markdown
Contributor

Migrates the Sync client to use the newly introduced unified checksum options. Also includes the necessary fallback logic to support users who are still using the deprecated options.

Note: This PR is stacked on top of #16261 and includes its commits. It should be merged after #16261.

@v-pratap
v-pratap requested review from a team as code owners July 16, 2026 13:15
@product-auto-label product-auto-label Bot added the api: storage Issues related to the Cloud Storage API. label Jul 16, 2026
@v-pratap v-pratap added the do not review Indicates a PR is not ready for review label Jul 16, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces UploadChecksumValidationOption and DownloadChecksumValidationOption using a new ChecksumAlgorithm enum to replace several deprecated checksum options and classes. It updates internal hash functions, validators, and tests to support these new options. The review feedback highlights that using the deprecated types in hash_function.cc, hash_validator.cc, and object_requests.cc will trigger -Wdeprecated-declarations compiler warnings, suggesting the inclusion of deprecation warning disable headers. Additionally, the reviewer recommends refactoring duplicated fallback logic for resolving checksum settings into a shared helper function to align with the repository style guide.

Comment thread google/cloud/storage/internal/hash_function.cc Outdated
Comment thread google/cloud/storage/internal/hash_validator.cc Outdated
Comment thread google/cloud/storage/internal/object_requests.cc Outdated
Comment thread google/cloud/storage/internal/hash_function.cc Outdated
@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.43750% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.27%. Comparing base (fd956fb) to head (c7928ae).

Files with missing lines Patch % Lines
google/cloud/storage/client_object_test.cc 96.72% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #16264      +/-   ##
==========================================
- Coverage   92.27%   92.27%   -0.01%     
==========================================
  Files        2218     2219       +1     
  Lines      206654   206686      +32     
==========================================
+ Hits       190699   190720      +21     
- Misses      15955    15966      +11     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@v-pratap
v-pratap force-pushed the unified-checksum-options-pr3 branch from d2b9add to 68aaf3e Compare July 17, 2026 05:58
v-pratap added a commit to v-pratap/google-cloud-cpp that referenced this pull request Jul 17, 2026
v-pratap added a commit to v-pratap/google-cloud-cpp that referenced this pull request Jul 17, 2026
@v-pratap
v-pratap force-pushed the unified-checksum-options-pr3 branch from eae7fbf to 59e1bf1 Compare July 17, 2026 08:53
@v-pratap
v-pratap force-pushed the unified-checksum-options-pr3 branch 2 times, most recently from 115a30d to 163523d Compare July 20, 2026 14:13
@v-pratap
v-pratap force-pushed the unified-checksum-options-pr3 branch from 163523d to 530f38b Compare July 20, 2026 14:25
@v-pratap v-pratap removed the do not review Indicates a PR is not ready for review label Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant