Skip to content

Eg add response methods#41879

Merged
billwert merged 9 commits into
Azure:mainfrom
billwert:eg-add-response-methods
Sep 20, 2024
Merged

Eg add response methods#41879
billwert merged 9 commits into
Azure:mainfrom
billwert:eg-add-response-methods

Conversation

@billwert

Copy link
Copy Markdown
Contributor

Add missing withResponse methods for sending/receiving with a RequestOptions.

Also ensure HTTP timeouts are set properly.

Add missing withResponse methods for sending/receiving with a `RequestOptions`.

Also ensure HTTP timeouts are set properly.
@azure-sdk

Copy link
Copy Markdown
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

com.azure:azure-messaging-eventgrid-namespaces

@billwert billwert force-pushed the eg-add-response-methods branch from b467dcf to 4ccd4b4 Compare September 17, 2024 19:44

@srnagar srnagar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@billwert billwert merged commit 4049c20 into Azure:main Sep 20, 2024
@billwert billwert deleted the eg-add-response-methods branch September 20, 2024 17:26
mssfang pushed a commit to mssfang/azure-sdk-for-java that referenced this pull request Oct 21, 2024
* Add WithResponse methods

Add missing withResponse methods for sending/receiving with a `RequestOptions`.

Also ensure HTTP timeouts are set properly.

* remove a couple extra methods

* make `addTimeoutToContext` helper method package-private

* remove extra comments

* fix comment

* just call the other method

* Add a little buffer to the http timeout in the default case.

* run tsp-client update to pick up formatting fixes

* add a comment about how fractional seconds are handled.
samvaity added a commit to samvaity/azure-sdk-for-java that referenced this pull request Apr 1, 2026
TypeSpec version gating restored for Azure Machine Learning, causing
SDK generation changes and potential customization drift in Java SDK.

Ref: Azure/azure-rest-api-specs#41879

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants