Skip to content

Conversation

@steveniemitz
Copy link
Contributor

This creates unnecessary allocations/work in the common case where the precondition is met.

@steveniemitz steveniemitz requested review from a team as code owners October 10, 2025 16:11
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: bigtable Issues related to the googleapis/java-bigtable API. labels Oct 10, 2025
@steveniemitz steveniemitz force-pushed the remove-string-formats branch from 5926cc3 to fd9da65 Compare October 10, 2025 16:15
@steveniemitz steveniemitz changed the title Don't use String.format in Preconditions messages fix: don't use String.format in Preconditions messages Oct 10, 2025
mutianf
mutianf previously approved these changes Oct 10, 2025
@steveniemitz steveniemitz force-pushed the remove-string-formats branch from fd9da65 to 698e1f3 Compare October 10, 2025 16:20
@mutianf mutianf merged commit 62a1812 into googleapis:main Oct 10, 2025
24 checks passed
@steveniemitz steveniemitz deleted the remove-string-formats branch October 10, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the googleapis/java-bigtable API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants