Skip to content

Conversation

@rakshith91
Copy link
Contributor

@rakshith91 rakshith91 commented Aug 3, 2021

Fixes #20015

  • Flatten the batch response
  • Add error to logs query result

@rakshith91
Copy link
Contributor Author

/azp run python - monitor - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rakshith91
Copy link
Contributor Author

/azp run python - monitor - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rakshith91
Copy link
Contributor Author

/azp run python - monitor - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rakshith91
Copy link
Contributor Author

/azp run python - monitor - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rakshith91
Copy link
Contributor Author

/azp run python - monitor - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

:keyword render: Any object.
:paramtype render: object
:keyword error: Any object.
:paramtype error: object
Copy link
Contributor

Choose a reason for hiding this comment

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

is this more of a result object or would a user create this in code? I ask because I think it would make more sense to document these as ivars if it's just a result object (edit: same for the type below)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fair enough - updating

),
LogsBatchQueryRequest(
query= "AppRequests | take 5",
query= "AppRequestss | take 5",
Copy link
Contributor

Choose a reason for hiding this comment

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

intentional change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yep - to show how to process an error

@rakshith91
Copy link
Contributor Author

/azp run python - monitor - tests

@rakshith91 rakshith91 enabled auto-merge (squash) August 3, 2021 17:45
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rakshith91
Copy link
Contributor Author

/azp run python - monitor - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rakshith91
Copy link
Contributor Author

/azp run python - monitor - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rakshith91
Copy link
Contributor Author

/check-enforcer override

@rakshith91 rakshith91 merged commit d01a468 into Azure:main Aug 3, 2021
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-python that referenced this pull request Aug 5, 2022
Dev advisor microsoft.advisor 2022 09 01 (Azure#20049)

* Adds base for updating Microsoft.Advisor from version stable/2020-01-01 to version 2022-09-01

* Updates readme

* Updates API version in new specs and examples

* Add duration and change examples

* Add predict api to version

* Delete feedback.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Flatten" body and tables in the logs batch query result. The goal is to make code like the following a bit simpler:

2 participants