Skip to content

feat(Spanner): Add the built in metrics feature#9055

Merged
bshaffer merged 23 commits into
mainfrom
builtin-metrics
Jun 9, 2026
Merged

feat(Spanner): Add the built in metrics feature#9055
bshaffer merged 23 commits into
mainfrom
builtin-metrics

Conversation

@Hectorhammett

Copy link
Copy Markdown
Collaborator

Adds the Client built in metrics for Spanner.

b/406082122

@product-auto-label product-auto-label Bot added the api: spanner Issues related to the Spanner API. label Mar 27, 2026
@Hectorhammett Hectorhammett marked this pull request as ready for review April 10, 2026 22:48
@Hectorhammett Hectorhammett requested a review from a team as a code owner April 10, 2026 22:48
@Hectorhammett Hectorhammett added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 10, 2026
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php Outdated
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php Outdated
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php Outdated
@Hectorhammett Hectorhammett removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label May 25, 2026
@Hectorhammett Hectorhammett added next release PRs to be included in the next release do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels May 29, 2026
@bshaffer bshaffer removed the next release PRs to be included in the next release label Jun 1, 2026
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php Outdated
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php Outdated
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php Outdated
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php Outdated
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php
Comment thread Spanner/src/Middleware/MetricsOperationMiddleware.php Outdated
Comment thread Spanner/src/Middleware/MetricsAttemptMiddleware.php Outdated
@Hectorhammett Hectorhammett removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 5, 2026
@bshaffer bshaffer added the next release PRs to be included in the next release label Jun 5, 2026
Comment thread Spanner/src/Middleware/MetricsOperationMiddleware.php
Comment thread Spanner/src/Operation.php Outdated
Comment thread Spanner/src/Operation.php Outdated
Comment thread Spanner/src/Operation.php Outdated
}
}

if (!$metricsClient instanceof MetricServiceClient) {

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.

NOTE: This could be updated to use an interface once we merge and release googleapis/gapic-generator-php#818

Comment thread Spanner/src/SpannerClient.php Outdated
@bshaffer bshaffer merged commit 6ae0728 into main Jun 9, 2026
39 checks passed
@bshaffer bshaffer deleted the builtin-metrics branch June 9, 2026 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the Spanner API. next release PRs to be included in the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants