Query Method Documentation Out of Date?

[ay-azara]

Library name and version

Azure.Monitory.Query Latest

Query/Question

https://learn.microsoft.com/en-us/dotnet/api/azure.monitor.query.logsqueryclient.queryresourceasync?view=azure-dotnet

Sorry if I'm missing something obvious but is this documentation out of date? QueryResourceAsync() and company return Response<IReadOnlyList<LogsQueryResult>> not Response<LogsQueryResult> so none of these examples are usable, as far as I can tell. At least, not without indexing the first result out, but are these methods supposed to return multiple results instead of just one?

Side note: If you have any examples of error handling that would also be cool

Environment

No response

[github-actions]

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @thomasp98296.

[github-actions]

Hello @ay-azara. I'm an AI assistant for the azure-sdk-for-net repository. I have some suggestions that you can try out while the team gets back to you.

• The current implementation of QueryResourceAsync() now returns a Response<IReadOnlyList<LogsQueryResult>> rather than a single LogsQueryResult. This change enables support for multiple query results—but it does mean that existing code examples may no longer work as written.

• The documentation at QueryResourceAsync API reference appears to be out of date, and we’re reviewing this discrepancy.

• In the meantime, you can update your code to work with the new return type (i.e. iterate over the collection in the IReadOnlyList<LogsQueryResult>), and we recommend filing feedback on the documentation if you haven’t already.

• We appreciate the feedback and are working to ensure our documentation examples align with the latest SDK changes.

The team will get back to you shortly, hopefully this helps in the meantime.

[jsquire]

//cc: @scottaddie

[jsquire]

Thank you for your feedback. Tagging and routing to the team member best able to assist.

[ay-azara]

Sorry, I must have been having a dunce moment when I opened this. I'm not very familiar with C# and it seems I was using Task<Response<IReadOnlyList<T>>> QueryResourceAsync<T> rather than Task<Response<LogsQueryResult>> QueryResourceAsync which is what I was looking for. The docs are fine the issue was me.