Skip to content
This repository was archived by the owner on May 8, 2026. It is now read-only.

tests: Remove unnecessary null check for protobuf lists - #2723

Merged
blakeli0 merged 3 commits into
mainfrom
blakeli0-patch-2
Dec 5, 2025
Merged

tests: Remove unnecessary null check for protobuf lists#2723
blakeli0 merged 3 commits into
mainfrom
blakeli0-patch-2

Conversation

@blakeli0

@blakeli0blakeli0 commented Dec 3, 2025

Copy link
Copy Markdown
Contributor

ErrorProne will complain ImpossibleNullComparison once protobuf runtime is upgraded to 4.x. See the downstream tests in sdk-platform-java.

@product-auto-labelproduct-auto-labelBot added size: s Pull request size is small. api: bigtable Issues related to the googleapis/java-bigtable API. labels Dec 3, 2025
@blakeli0blakeli0 changed the title tests: Remove unnecessary null check for protobuf gen codetests: Remove unnecessary null check for a protobuf listDec 4, 2025
@blakeli0blakeli0 changed the title tests: Remove unnecessary null check for a protobuf listtests: Remove unnecessary null check for protobuf listsDec 4, 2025
@blakeli0
blakeli0 marked this pull request as ready for review December 4, 2025 19:12
@blakeli0
blakeli0 requested review from a teamDecember 4, 2025 19:12
@blakeli0blakeli0 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 4, 2025
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 4, 2025
@blakeli0
blakeli0 merged commit cfa4708 into mainDec 5, 2025
26 of 27 checks passed
@blakeli0
blakeli0 deleted the blakeli0-patch-2 branch December 5, 2025 14:39
diegomarquezp pushed a commit that referenced this pull request Dec 16, 2025
ErrorProne will complain ImpossibleNullComparison once protobuf runtime is upgraded to 4.x. See the downstream tests in sdk-platform-java.
blakeli0 added a commit that referenced this pull request Dec 17, 2025
* deps: update sdk-platform-java-config to 3.55.0-rc1
* chore: generate libraries at Mon Dec 15 17:11:37 UTC 2025
* chore: follow error prone suggestion (non-nullable return value)
* chore: remove redundant condition
* Revert "chore: remove redundant condition"
This reverts commit 6b3bc19.
* Revert "chore: follow error prone suggestion (non-nullable return value)"
This reverts commit e841b54.
* tests: Remove unnecessary null check for protobuf lists (#2723)
ErrorProne will complain ImpossibleNullComparison once protobuf runtime is upgraded to 4.x. See the downstream tests in sdk-platform-java.
---------
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
Co-authored-by: Blake Li <blakeli@google.com>
blakeli0 added a commit that referenced this pull request Feb 3, 2026
* tests: Remove unnecessary null check for protobuf lists (#2723)
ErrorProne will complain ImpossibleNullComparison once protobuf runtime is upgraded to 4.x. See the downstream tests in sdk-platform-java.
* chore: add release-please config for protobuf-4.x (#2722)
* chore: add release-please config for protobuf-4.x
* chore: generate libraries at Wed Dec 3 18:40:37 UTC 2025
* chore: enable manifest option in release config
---------
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
* deps: update shared dependencies (#2734)
* chore(main): release 2.70.1-SNAPSHOT (#2715)
* chore(main): release 2.70.1-SNAPSHOT
* chore: generate libraries at Fri Dec 12 14:04:07 UTC 2025
---------
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
* chore(main): release 2.70.1 (#2735)
* chore(main): release 2.70.1
* chore: generate libraries at Fri Dec 12 15:48:36 UTC 2025
---------
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
* fix(bigtable): add handling for gauge metrics (#2719)
* fix(bigtable): add handling for gauge metrics
* add it
* chore: generate libraries at Tue Nov 25 20:15:38 UTC 2025
---------
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
* chore(main): release 2.70.2-SNAPSHOT (#2736)
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* feat: Add CSM for batch write flow control (#2685)
* test: Add missing test table cleanup for MaterializedViewIT (#2721)
Change-Id: I1281568894ac2d04f0aafe0aa7f4efa7570ae58a
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [ ] Make sure to open an issue as a [bug/issue](https://togithub.com/googleapis/java-bigtable/issues/new/choose) before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)
- [ ] Rollback plan is reviewed and LGTMed
- [ ] All new data plane features have a completed end to end testing plan
Fixes #<issue_number_goes_here> ☕️
If you write sample code, please follow the [samples format](
https://togithub.com/GoogleCloudPlatform/java-docs-samples/blob/main/SAMPLE_FORMAT.md).
* tests: Migrate usage of GeneratedMessageV3 to Message (#2745)
* tests: Migrate usage of GeneratedMessageV3 to Message
* chore: generate libraries at Mon Dec 22 22:29:53 UTC 2025
---------
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
* fix: create stub with BigtableClientContext so otels are closed (#2747)
* fix: create stub with BigtableClientContext so otels are closed
* address comment
* test: Reduce ExecuteQueryIT flakiness with a retry loop (#2748)
Change-Id: I4171e5b11d2278a8b3dc19d2dec3f03d784759ce
* fix: use the same background executor in otel reader and monitoring c… (#2746)
* fix: use the same background executor in otel reader and monitoring client
* add public api doc
* fix tests
* close background executor
* fix test
* address comments
* fix
* deps: update shared dependencies (#2752)
* fix: update BigtableChannelPool to use the background executor (#2753)
* deps: update dependency com.google.cloud:gapic-libraries-bom to v1.76.0 (#2754)
* chore(main): release 2.71.0 (#2737)
* chore(main): release 2.71.0
* chore: generate libraries at Thu Jan 15 16:06:19 UTC 2026
---------
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
---------
Co-authored-by: Diego Marquez <diegomarquezp@google.com>
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
Co-authored-by: Mend Renovate <bot@renovateapp.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Sushan Bhattarai <sushantsusan@google.com>
Co-authored-by: Weihan Kong <kongweihan@users.noreply.github.com>
Co-authored-by: Lixia Chen <trollyxia@gmail.com>
Co-authored-by: Mattie Fu <mattiefu@google.com>
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigtableIssues related to the googleapis/java-bigtable API.size: sPull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@blakeli0@mutianf@yoshi-kokoro@cloud-java-bot