FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(bigtable): add handling for gauge metrics by sushanb · Pull Request #2719 · googleapis/java-bigtable · GitHub

This repository was archived by the owner on May 8, 2026. It is now read-only.
/ java-bigtable Public archive

fix(bigtable): add handling for gauge metrics - #2719

Merged
sushanb merged 3 commits into
mainfrom
sushanb_1
Dec 12, 2025
Merged

fix(bigtable): add handling for gauge metrics#2719
sushanb merged 3 commits into
mainfrom
sushanb_1

Conversation

sushanb commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

Field points[0].interval.start_time had an invalid value of "2025-11-25T11:10:06.164418-08:00": The start time must be equal to the end time (2025-11-25T11:11:06.164633-08:00) for the gauge metric 'bigtable.googleapis.com/internal/client/grpc/subchannel/open_connections'

sushanb requested review from a team November 25, 2025 19:31
product-auto-label Bot added size: s Pull request size is small. api: bigtable Issues related to the googleapis/java-bigtable API. labels Nov 25, 2025
sushanb requested a review from mutianf November 25, 2025 19:32
sushanb added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 25, 2025
yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 25, 2025
cloud-java-bot requested a review from a team November 25, 2025 19:34
sushanb force-pushed the sushanb_1 branch 2 times, most recently from d21e526 to c59bfb6 Compare November 25, 2025 20:15
Timestamp endTimestamp = Timestamps.fromNanos(pointData.getEpochNanos());
Timestamp startTimestamp;

if (kind == GAUGE) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

nit: can we extract this to a helper method? handleGaugeMetric() or somehing

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

FYI, this won't remain very long in 2 versions as we will move all metrics to cloud.BigtableClient schema. so let's put it here for now temporarily.

sushanb merged commit 87aa4d5 into main Dec 12, 2025
27 of 28 checks passed
sushanb deleted the sushanb_1 branch December 12, 2025 17:02
diegomarquezp pushed a commit that referenced this pull request Dec 15, 2025
* 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>
diegomarquezp added a commit that referenced this pull request Dec 15, 2025
* chore: sync with last release from main branch

* 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: generate libraries at Mon Dec 15 17:47:00 UTC 2025

* chore: fix versiontest

* chore: remove unwanted escape sequence

* chore: remove unintended match character

* chore(main): release 2.70.2-SNAPSHOT (#2736)

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

* chore: generate libraries at Mon Dec 15 21:32:02 UTC 2025

---------

Co-authored-by: Sushan Bhattarai <sushantsusan@google.com>
Co-authored-by: cloud-java-bot <cloud-java-bot@google.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.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>
sushanb added a commit to googleapis/google-cloud-go that referenced this pull request Feb 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

4 participants


Back | FazBrowse Home | New Git URL