Visitar URL original
feat: idle channel eviction by nicholsl · Pull Request #2651 · googleapis/java-bigtable · GitHub
Skip to content
This repository was archived by the owner on Aug 30, 2026. It is now read-only.

feat: idle channel eviction - #2651

Merged
igorbernstein2 merged 27 commits into
googleapis:mainfrom
nicholsl:feat/idle-channel-eviction
Sep 10, 2025
Merged

feat: idle channel eviction#2651
igorbernstein2 merged 27 commits into
googleapis:mainfrom
nicholsl:feat/idle-channel-eviction

Conversation

@nicholsl

@nicholsl nicholsl commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

Change-Id: I62fe152c293438bf64b657b5b1fe795e22ce9c85

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 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.

@nicholsl
feat/idle-channel-eviction
20e012d 
Change-Id: I62fe152c293438bf64b657b5b1fe795e22ce9c85
@nicholsl
nicholsl requested review from a team August 1, 2025 14:58
@product-auto-label product-auto-label Bot added size: l Pull request size is large. api: bigtable Issues related to the googleapis/java-bigtable API. labels Aug 1, 2025
@nicholsl
create noop channel primer
d2b6129 
Change-Id: I8d4212aad0ca7613b6c33d69d925671a090d1609
@nicholsl
no-op channel primer changes
333e9e9 
Change-Id: I9ad5eeaacb02ace9ba3cf7c09b6846dcbc298fb8
@nicholsl
update ignored diffs
4ddefce 
Change-Id: I04698ba82b95c82301e4e9f436401698675d7ea9
@nicholsl
nicholsl requested a review from a team August 27, 2025 15:22
cloud-java-bot and others added 7 commits August 27, 2025 11:25
@cloud-java-bot @nicholsl
chore: Update generation configuration at Thu Aug 7 02:49:23 UTC 2025 (…
ba9e039 
…googleapis#2649)

* chore: Update generation configuration at Thu Jul 31 02:47:07 UTC 2025

* chore: Update generation configuration at Fri Aug  1 02:54:57 UTC 2025

* chore: Update generation configuration at Sat Aug  2 02:42:43 UTC 2025

* chore: generate libraries at Sat Aug  2 02:43:12 UTC 2025

* chore: Update generation configuration at Tue Aug  5 02:50:25 UTC 2025

* chore: generate libraries at Tue Aug  5 02:50:52 UTC 2025

* chore: Update generation configuration at Wed Aug  6 02:48:20 UTC 2025

* chore: generate libraries at Wed Aug  6 02:48:53 UTC 2025

* chore: Update generation configuration at Thu Aug  7 02:49:23 UTC 2025
@mutianf @nicholsl
deps: update shared dependencies (googleapis#2654)
3646452
@release-please @cloud-java-bot @nicholsl
chore(main): release 2.64.0 (googleapis#2652)
b741869 
* chore(main): release 2.64.0

* chore: generate libraries at Fri Aug  8 14:28:59 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>
@sushanb @nicholsl
feat(bigtable): lower the value for max rpc channels as channel resiz…
bafedf9 
…e is slow (1m, 2 channel) (googleapis#2656)

…conservative

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).
@release-please @nicholsl
chore(main): release 2.64.1-SNAPSHOT (googleapis#2655)
cbdb4e2 
🤖 I have created a release *beep* *boop*
---


### Updating meta-information for bleeding-edge SNAPSHOT release.

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
@release-please @cloud-java-bot @nicholsl
chore(main): release 2.65.0 (googleapis#2657)
0422092 
* chore(main): release 2.65.0

* chore: generate libraries at Tue Aug 12 16:25:49 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>
@release-please @nicholsl
chore(main): release 2.65.1-SNAPSHOT (googleapis#2658)
171ac40 
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
@generated-files-bot

generated-files-bot Bot commented Aug 27, 2025

Copy link
Copy Markdown

Warning: This pull request is touching the following templated files:

  • google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/admin/v2/BaseBigtableInstanceAdminClient.java
  • google-cloud-bigtable/src/test/java/com/google/cloud/bigtable/admin/v2/BaseBigtableInstanceAdminClientTest.java
  • proto-google-cloud-bigtable-admin-v2/src/main/java/com/google/bigtable/admin/v2/AppProfile.java
  • proto-google-cloud-bigtable-admin-v2/src/main/java/com/google/bigtable/admin/v2/AppProfileOrBuilder.java
  • proto-google-cloud-bigtable-admin-v2/src/main/java/com/google/bigtable/admin/v2/Instance.java
  • proto-google-cloud-bigtable-admin-v2/src/main/java/com/google/bigtable/admin/v2/InstanceOrBuilder.java
  • proto-google-cloud-bigtable-admin-v2/src/main/java/com/google/bigtable/admin/v2/InstanceProto.java
  • proto-google-cloud-bigtable-admin-v2/src/main/proto/google/bigtable/admin/v2/instance.proto
  • samples/snapshot/pom.xml

nicholsl and others added 4 commits August 27, 2025 11:46
@nicholsl
Merge branch 'googleapis:main' into feat/idle-channel-eviction
5b619b1
@nicholsl
more noop channel primer changes
862135b 
Change-Id: I3baa8e93bc614efe21f1159a708be36881120e34
@nicholsl
remove outdated comments
c0082cf 
Change-Id: I11e8da901a93a49009caf4caf0385fd5e3c8091b
@nicholsl
cleanup
7a007bf 
Change-Id: Iaa30338d04ba846469fc6152e0683e53573c40ee
mutianf
mutianf reviewed Aug 28, 2025
View reviewed changes
Comment thread ...e-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/stub/NoOpChannelPrimer.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...le-cloud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/BigtableChannelPool.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...le-cloud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/BigtableChannelPool.java
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
nicholsl and others added 2 commits August 28, 2025 16:04
@nicholsl
pr feedback
beda46e 
Change-Id: Ie0fde53cd17b1feeaaffd15382e936a46d84c5c1
@nicholsl
Merge branch 'googleapis:main' into feat/idle-channel-eviction
dff1361
@mutianf mutianf changed the title feat/idle-channel-eviction feat: idle channel eviction Sep 2, 2025
mutianf
mutianf previously approved these changes Sep 2, 2025
View reviewed changes
@nicholsl
fix: lint
e6cab0c 
Change-Id: Ib8c6b56569c17997a126065a6fa00e8bb1604494
mutianf
mutianf previously approved these changes Sep 2, 2025
View reviewed changes
@mutianf mutianf added the kokoro:run Add this label to force Kokoro to re-run the tests. label Sep 2, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Sep 2, 2025
@neilw4 neilw4 mentioned this pull request Sep 4, 2025
Merged
igorbernstein2
igorbernstein2 suggested changes Sep 4, 2025
View reviewed changes
Comment thread ...le-cloud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/BigtableChannelPool.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java
Comment thread google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPrimer.java Outdated
Comment thread ...bigtable/src/test/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthCheckerTest.java
Comment thread google-cloud-bigtable/src/test/java/com/google/cloud/bigtable/gaxx/grpc/HealthChecker.java
@nicholsl
fix: pr feedback
c77e876 
Change-Id: Ic5f18450c5f60bcf5473e6890b41b006821ddf9d
igorbernstein2
igorbernstein2 suggested changes Sep 8, 2025
View reviewed changes
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...le-cloud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/BigtableChannelPool.java Outdated
Comment thread ...le-cloud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/BigtableChannelPool.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
nicholsl and others added 3 commits September 9, 2025 16:46
@nicholsl
fix: feedback
98fe29b 
Change-Id: Ib7f24524b6050ec11ca18e5dc624ca8513f99ec8
@nicholsl
fix: lint
69aeeef 
Change-Id: I78712b99a975fb6ec9defcdb6812e955df555e1f
@nicholsl
Merge branch 'googleapis:main' into feat/idle-channel-eviction
52e1cb7
igorbernstein2
igorbernstein2 reviewed Sep 9, 2025
View reviewed changes
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
Comment thread ...oud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPoolHealthChecker.java Outdated
@nicholsl
move history pruning
32a7ac0 
Change-Id: Ibeae2ce0b790ef2731be541a70fa829e220fa4a8
@nicholsl
fix: nits
580d5de 
Change-Id: Ic812e3523a9f29dc612fbdf58ee8f9dbb8a283f4
@igorbernstein2 igorbernstein2 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 9, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 9, 2025
igorbernstein2
igorbernstein2 previously approved these changes Sep 10, 2025
View reviewed changes
mutianf
mutianf reviewed Sep 10, 2025
View reviewed changes
Comment thread google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPrimer.java Outdated
Comment thread google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/gaxx/grpc/ChannelPrimer.java Outdated
@nicholsl @mutianf
Apply suggestion from @mutianf
111582c 
Co-authored-by: Mattie Fu <mattiefu@google.com>
@mutianf mutianf added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 10, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 10, 2025
@igorbernstein2
igorbernstein2 merged commit 70c05c9 into googleapis:main Sep 10, 2025
23 of 25 checks passed
@release-please release-please Bot mentioned this pull request Sep 24, 2025
Merged
gcf-merge-on-green Bot pushed a commit that referenced this pull request Sep 25, 2025
@release-please
chore(main): release 2.67.0 (#2678)
f546838 
🤖 I have created a release *beep* *boop*
---


## [2.67.0](https://togithub.com/googleapis/java-bigtable/compare/v2.66.0...v2.67.0) (2025-09-24)


### Features

* Idle channel eviction ([#2651](https://togithub.com/googleapis/java-bigtable/issues/2651)) ([70c05c9](https://togithub.com/googleapis/java-bigtable/commit/70c05c9c09a63c53818384d2a66c622c9b95e00e))
* Load balancing options for BigtableChannelPool ([#2667](https://togithub.com/googleapis/java-bigtable/issues/2667)) ([5adaa84](https://togithub.com/googleapis/java-bigtable/commit/5adaa84d80df08779da7c36a50de4632049cfe96))


### Bug Fixes

* Add missing break; to PROTO and ENUM value type check ([#2672](https://togithub.com/googleapis/java-bigtable/issues/2672)) ([337e432](https://togithub.com/googleapis/java-bigtable/commit/337e4325f6cb5d11309ec5f33550d47d97cbe3c3))
* Remove beta api annotation for query paginator ([#2660](https://togithub.com/googleapis/java-bigtable/issues/2660)) ([f68a1fa](https://togithub.com/googleapis/java-bigtable/commit/f68a1fae49b701d1fb9942e2af2fa84a1e5b508a))


### Dependencies

* Update shared dependencies ([#2679](https://togithub.com/googleapis/java-bigtable/issues/2679)) ([a5b8260](https://togithub.com/googleapis/java-bigtable/commit/a5b82609c365ae4792ed822e59039c1a046ef3ff))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
@release-please release-please Bot mentioned this pull request Dec 15, 2025
Merged
@lcian lcian mentioned this pull request Mar 6, 2026
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers
@mutianf mutianf mutianf approved these changes
@igorbernstein2 igorbernstein2 igorbernstein2 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@nicholsl @mutianf @igorbernstein2 @yoshi-kokoro @cloud-java-bot @sushanb