Skip to content

feat(storage): Add method to disable checksums on read object#4993

Draft
vsharonlynn wants to merge 6 commits intogoogleapis:mainfrom
vsharonlynn:feat/storage-disable-checksum
Draft

feat(storage): Add method to disable checksums on read object#4993
vsharonlynn wants to merge 6 commits intogoogleapis:mainfrom
vsharonlynn:feat/storage-disable-checksum

Conversation

@vsharonlynn
Copy link
Contributor

For issue #4285.

@product-auto-label product-auto-label bot added the api: storage Issues related to the Cloud Storage API. label Mar 12, 2026
@codecov
Copy link

codecov bot commented Mar 12, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.77%. Comparing base (4e48dac) to head (565014b).

Files with missing lines Patch % Lines
src/storage/src/storage/read_object.rs 50.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4993      +/-   ##
==========================================
- Coverage   92.82%   92.77%   -0.06%     
==========================================
  Files         225      225              
  Lines        8665     8673       +8     
==========================================
+ Hits         8043     8046       +3     
- Misses        622      627       +5     

☔ View full report in Codecov by Sentry.
📢 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.

Copy link
Collaborator

@coryan coryan left a comment

Choose a reason for hiding this comment

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

Drive-by, as I may not be available tomorrow.

@vsharonlynn vsharonlynn force-pushed the feat/storage-disable-checksum branch from 51494ca to a4acb16 Compare March 13, 2026 03:33
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.

3 participants