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.

Sign up for GitHub

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

Jump to bottom

Investigate the passive timeout strategy for v8 #1133

Closed
martintmk opened this issue Apr 15, 2023 · 2 comments
Closed

Investigate the passive timeout strategy for v8 #1133

martintmk opened this issue Apr 15, 2023 · 2 comments
Labels
up-for-grabs v8 Issues related to the new version 8 of the Polly library.
Milestone
v8.0.0

Comments

@martintmk
Copy link
Contributor

The timeout strategy in v8 only supports optimistic timeouts. We should investigate whatever adding the pessimistic timeout is something we want to support in v8 . If the answer is yes we should add support for it.

Using pessimistic timeout can lead to system overloads when a bunch of non-cancellable actions are being executed in the background without the user knowing about it.
@martintmk martintmk added up-for-grabs v8 Issues related to the new version 8 of the Polly library. labels Apr 15, 2023
@martintmk martintmk added this to the v8.0.0 milestone Apr 15, 2023
@martintmk martintmk mentioned this issue Jun 15, 2023
Closed
17 tasks
@github-actions
Copy link
Contributor

github-actions bot commented Jul 1, 2023

This issue is stale because it has been open for 60 days with no activity. It will be automatically closed in 14 days if no further updates are made.

@github-actions github-actions bot added the stale Stale issues or pull requests label Jul 1, 2023
@martincostello martincostello removed the stale Stale issues or pull requests label Jul 1, 2023
@martintmk
Copy link
Contributor Author

Added a note to the migration guide that the pessimistic timeout is not available - #1622

Closing the issue as currently pessimistic timeout will be v7 only. Based on the feedback we can create a new one or re-open this one.

@github-project-automation github-project-automation bot moved this to Done in Polly v8 Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
up-for-grabs v8 Issues related to the new version 8 of the Polly library.
Projects
No open projects
Polly v8
Status: Done
Milestone
v8.0.0
Development

No branches or pull requests
2 participants
@martincostello @martintmk