Skip to content

feat(bigtable): Record client_blocking_latencies - #12698

Merged
bhshkh merged 9 commits into
googleapis:mainfrom
bhshkh:feature/cbt-client-blocking-latencies-pr
Aug 15, 2025
Merged

feat(bigtable): Record client_blocking_latencies#12698
bhshkh merged 9 commits into
googleapis:mainfrom
bhshkh:feature/cbt-client-blocking-latencies-pr

Conversation

@bhshkh

@bhshkh bhshkh commented Aug 12, 2025

Copy link
Copy Markdown
Contributor

This PR adds client_blocking_latencies metrics. More about the metric can be found at https://cloud.google.com/bigtable/docs/client-side-metrics-descriptions#client-blocking-latencies

@bhshkh
bhshkh requested review from a team August 12, 2025 16:15
@product-auto-label product-auto-label Bot added the api: bigtable Issues related to the Bigtable API. label Aug 12, 2025
@danowar2

Copy link
Copy Markdown

Think I need a new available for Data

@bhshkh
bhshkh force-pushed the feature/cbt-client-blocking-latencies-pr branch from 96b7f73 to c1aa6eb Compare August 12, 2025 17:31
@bhshkh
bhshkh requested a review from jba as a code owner August 12, 2025 17:31
@bhshkh
bhshkh requested review from a team August 12, 2025 17:31
@bhshkh
bhshkh requested a review from shollyman as a code owner August 12, 2025 17:31
@bhshkh
bhshkh requested review from a team and cindy-peng August 12, 2025 17:31
@snippet-bot

snippet-bot Bot commented Aug 12, 2025

Copy link
Copy Markdown

No region tags are edited in this PR.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@bhshkh
bhshkh force-pushed the feature/cbt-client-blocking-latencies-pr branch from e21c9ab to 7772b51 Compare August 12, 2025 17:40
@bhshkh
bhshkh removed request for a team August 12, 2025 17:41
@bhshkh
bhshkh removed request for a team, cindy-peng, jba and shollyman August 12, 2025 17:41
@bhshkh
bhshkh marked this pull request as draft August 12, 2025 17:42
@bhshkh
bhshkh force-pushed the feature/cbt-client-blocking-latencies-pr branch from 7772b51 to 7c4d669 Compare August 12, 2025 17:43
Comment thread bigtable/metrics.go Outdated
Comment thread bigtable/metrics.go Outdated
Comment thread bigtable/metrics.go Outdated
@bhshkh
bhshkh marked this pull request as ready for review August 13, 2025 20:33
sushanb
sushanb previously approved these changes Aug 14, 2025
Comment thread bigtable/metrics.go Outdated
mutianf
mutianf previously approved these changes Aug 14, 2025
@bhshkh
bhshkh dismissed stale reviews from mutianf and sushanb via 39ba551 August 14, 2025 19:18
@bhshkh
bhshkh enabled auto-merge (squash) August 14, 2025 19:18
@bhshkh
bhshkh merged commit 5f03fb3 into googleapis:main Aug 15, 2025
11 checks passed
@bhshkh
bhshkh deleted the feature/cbt-client-blocking-latencies-pr branch August 15, 2025 19:20
@bhshkh bhshkh linked an issue Sep 30, 2025 that may be closed by this pull request
@bhshkh bhshkh linked an issue Sep 30, 2025 that may be closed by this pull request
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: advanced metrics

4 participants