Rename query APIs in Logs and Metrics Client #17852

srnagar · 2021-09-24T20:36:22Z

To prepare for supporting resource-centric logs queries, make the following name changes to the query APIs in Logs and Metrics clients.

queryWorkspace(workspaceId: string, query: string, timespan: TimeInterval, options?: LogsQueryOptions): Promise<LogsQueryResult>;

queryWorkspace(resourceUri: string, metricNames: string[], options?: MetricsQueryOptions): Promise<MetricsQueryResult>;

KarishmaGhiya · 2021-09-27T07:37:56Z

@srnagar isn't that TimeInterval supposed to be renamed to QueryTimeInterval?

rakshith91 · 2021-09-27T18:20:12Z

@srnagar isn't that TimeInterval supposed to be renamed to QueryTimeInterval?

looks like java changed it to QueryTimeInterval

KarishmaGhiya · 2021-09-30T07:50:41Z

Addressed in PR - #17873

srnagar added the Monitor - Query label Sep 24, 2021

srnagar added this to the [2021] October milestone Sep 24, 2021

srnagar assigned KarishmaGhiya Sep 24, 2021

KarishmaGhiya mentioned this issue Sep 27, 2021

Api Consistency Review feedback #17873

Merged

KarishmaGhiya closed this as completed Sep 30, 2021

github-actions bot locked and limited conversation to collaborators Apr 11, 2023

scottaddie added Monitor Monitor, Monitor Ingestion, Monitor Query and removed Monitor - Query labels Sep 28, 2023

Provide feedback