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

timeout improvements #927

Merged
merged 2 commits into from
Nov 20, 2023
Merged

timeout improvements #927

merged 2 commits into from
Nov 20, 2023

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Nov 20, 2023

  • delete unused TimeoutWatcher implementation
  • remove the concept of TimeoutWatcher

@Ladicek
delete unused TimeoutWatcher implementation
1103c92
@Ladicek
remove the concept of TimeoutWatcher
2860100 
The `TimeoutWatcher` predates and can easily be implemented in terms of
the `Timer`. This commit removes the `TimeoutWatcher` concept entirely,
because the `Timer` can be used as a replacement rather easily. There's
no need to have the extra layer of abstraction.
@Ladicek Ladicek added this to the 6.3.0 milestone Nov 20, 2023
@Ladicek Ladicek mentioned this pull request Nov 20, 2023
Closed
@Ladicek Ladicek merged commit 40cf278 into smallrye:main Nov 20, 2023
@Ladicek Ladicek deleted the timeout-improvements branch November 20, 2023 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
6.3.0
Development

Successfully merging this pull request may close these issues.
1 participant
@Ladicek