Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unnecessary async state machine from DefaultHttpRetryHandler #678

Merged
merged 2 commits into from Apr 26, 2023

Conversation

stephentoub
Copy link
Member

Motivation and Context

Small bit of unnecessary overhead.

Description

Removes one or more allocations every time it needs to delay.

Contribution Checklist

@github-actions github-actions bot added .NET Issue or Pull requests regarding .NET code kernel.core labels Apr 26, 2023
@lemillermicrosoft lemillermicrosoft enabled auto-merge (squash) April 26, 2023 22:37
@lemillermicrosoft lemillermicrosoft added the PR: ready to merge PR has been approved by all reviewers, and is ready to merge. label Apr 26, 2023
@lemillermicrosoft lemillermicrosoft merged commit 4e925b8 into microsoft:main Apr 26, 2023
11 checks passed
dluc pushed a commit that referenced this pull request Apr 29, 2023
…678)

### Motivation and Context

Small bit of unnecessary overhead.

### Description

Removes one or more allocations every time it needs to delay.
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
…icrosoft#678)

### Motivation and Context

Small bit of unnecessary overhead.

### Description

Removes one or more allocations every time it needs to delay.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.NET Issue or Pull requests regarding .NET code PR: ready to merge PR has been approved by all reviewers, and is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants