Skip to content

Conversation

@bhshkh
Copy link
Contributor

@bhshkh bhshkh commented Sep 30, 2025

Fix Bigtable test timeout in TestBatchingExport

The TestBatchingExport/1199_metrics test was timing out after 45 minutes.
This was caused by the exportTimeSeries function sending metric batches sequentially. With 1199 metrics, this resulted in 6 separate, synchronous gRPC calls, which caused a hang.

This change parallelizes the exportTimeSeries function to send metric batches concurrently using goroutines. This resolves the timeout and improves performance.

Fixes: #12920

@bhshkh bhshkh requested review from a team as code owners September 30, 2025 20:03
@product-auto-label product-auto-label bot added the api: bigtable Issues related to the Bigtable API. label Sep 30, 2025
@bhshkh bhshkh linked an issue Sep 30, 2025 that may be closed by this pull request
@bhshkh bhshkh enabled auto-merge (squash) September 30, 2025 20:18
@bhshkh bhshkh merged commit bafd691 into googleapis:main Oct 10, 2025
11 checks passed
@bhshkh bhshkh deleted the perf/cbt-async-create-timeseries branch October 17, 2025 19:32
bhshkh added a commit that referenced this pull request Dec 15, 2025
PR created by the Librarian CLI to initialize a release. Merging this PR
will auto trigger a release.

Librarian Version: v0.7.0
Language Image:
us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/librarian-go@sha256:718167d5c23ed389b41f617b3a00ac839bdd938a6bd2d48ae0c2f1fa51ab1c3d
<details><summary>bigtable: 1.41.0</summary>

##
[1.41.0](bigtable/v1.40.1...bigtable/v1.41.0)
(2025-12-15)

### Features

* add PeerInfo proto in Bigtable API (PiperOrigin-RevId: 829585900)
([185951b](185951b3))

* precompute featureflags in client and reuse (#13297)
([3b3253a](3b3253ac))

* Add experimental Bigtable connection pool with custom load balancing
strategy. (#12882)
([7aa9612](7aa96127))

* Enable ALTS hard bound token in Bigtable w/ direct access (#13153)
([9023934](90239341))

* Enable routing cookie and attempt headers for enhanced retries
(#12964)
([96cfd47](96cfd47a))

### Bug Fixes

* fix project id sent in otel (#13286)
([4865868](4865868c))

* screaming uppercase metric status (#13484)
([b35ee8f](b35ee8fd))

### Performance Improvements

* parallelize the exportTimeSeries function (#13004)
([bafd691](bafd691d))

</details>
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 Bigtable API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bigtable: Client side metrics performance improvements bigtable: TestBatchingExport failed

2 participants