feat: API regeneration: 20260520T070127Z#15622
Conversation
…c5fdd feat: add a AggregateQueryStats API PiperOrigin-RevId: 917888205 Source-Link: googleapis/googleapis@d4c5fdd
|
Here is the summary of changes. You are about to add 2 region tags.
This comment is generated by snippet-bot.
|
|
Pull request diff results Diff level: Minor Comparing with previous NuGet package Diff level: Minor Finished comparisons for Google.Cloud.DatabaseCenter.V1Beta |
There was a problem hiding this comment.
Code Review
This pull request introduces the AggregateQueryStats and AggregateQueryStatsAsync methods to the DatabaseCenterClient, providing database resource query execution statistics. The changes include the necessary client implementation, gRPC service updates, and generated code snippets. Feedback was provided regarding a potential naming mismatch in ServiceResourceNames.g.cs, where the partial class QueryStats should likely be renamed to QueryStatsInfo to align with the message type returned by the new API methods.
|
|
||
| namespace Google.Cloud.DatabaseCenter.V1Beta | ||
| { | ||
| public partial class QueryStats |
There was a problem hiding this comment.
The partial class is named QueryStats, but the AggregateQueryStats method returns a sequence of QueryStatsInfo objects (as seen in DatabaseCenterClient.g.cs and snippet_metadata_google.cloud.databasecenter.v1beta.json). If the ResourceTypeAsResourceName property was intended to be available on the items returned by AggregateQueryStats, this partial class should be named QueryStatsInfo to match the message type.
public partial class QueryStatsInfo
Changes in this PR