Skip to content

Conversation

@sushanb
Copy link
Contributor

@sushanb sushanb commented Nov 25, 2025

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 sushanb requested review from a team as code owners November 25, 2025 19:31
@product-auto-label 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 sushanb requested a review from mutianf November 25, 2025 19:32
@sushanb sushanb added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 25, 2025
@yoshi-kokoro 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 cloud-java-bot requested a review from a team as a code owner November 25, 2025 19:34
@sushanb 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
Contributor

Choose a reason for hiding this comment

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

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

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