KEMBAR78
chore: add Hasher#defaultHasher() by BenWhitehead · Pull Request #3173 · googleapis/java-storage · GitHub
Skip to content

Conversation

@BenWhitehead
Copy link
Collaborator

By default, it will be the same as Hasher.enabled(). If running on java < 9, a log message will be logged making it apparent there isn't a fast implementation of crc32c.

A system property com.google.cloud.storage.Hasher.default can be set to "disabled" to instead make the default Hasher.noop().

By default, it will be the same as Hasher.enabled(). If running on java < 9, a log message will be logged making it apparent there isn't a fast implementation of crc32c.

A system property `com.google.cloud.storage.Hasher.default` can be set to "disabled" to instead make the default Hasher.noop().
@BenWhitehead BenWhitehead requested a review from a team as a code owner June 26, 2025 18:42
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: storage Issues related to the googleapis/java-storage API. labels Jun 26, 2025
@BenWhitehead BenWhitehead added this to the e2e-crc32c-media milestone Jun 30, 2025
Copy link
Contributor

@cojenco cojenco left a comment

Choose a reason for hiding this comment

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

LGTM, had a GVC review session with Ben

BenWhitehead added a commit that referenced this pull request Jul 23, 2025
Merge train for

* chore: cleanup Crc32cValue and associated Hasher operations #3167
* chore: add Hasher#defaultHasher() #3173
* chore: add UnifiedOpts.Opts#getHasher() to return a hasher relative to the set of Opts defined for an operation. #3174
* feat: add default end-to-end crc32c checksumming for several upload methods via grpc transport #3176
* feat: add default end-to-end checksumming for JournalingBlobWriteSessionConfig #3180
* fix: give user provided checksum precondition priority for Storage#create methods that accept byte[] #3182
* fix: move crc32c computation before writing to disk for BufferToDiskThenUpload BlobWriteSession config #3187
@BenWhitehead
Copy link
Collaborator Author

merged in #3197

@BenWhitehead BenWhitehead deleted the checksumming/03/default-hasher branch July 23, 2025 18:14
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 googleapis/java-storage API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants