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

Scheduler: clarify what happens if scheduler is not started but method declared on Scheduler API is invoked #44101

Closed
mkouba opened this issue Oct 25, 2024 · 2 comments · Fixed by #44230
Closed

Scheduler: clarify what happens if scheduler is not started but method declared on Scheduler API is invoked #44101

mkouba opened this issue Oct 25, 2024 · 2 comments · Fixed by #44230
Assignees
@mkouba
Labels
area/scheduler kind/enhancement New feature or request
Milestone
3.17.0.CR1

Comments

@mkouba
Copy link
Contributor

mkouba commented Oct 25, 2024

Description

I think that all methods (except for isRunning()) should throw an UnsupportedOperationException.

Implementation ideas

No response
@mkouba mkouba added the kind/enhancement New feature or request label Oct 25, 2024
@mkouba mkouba self-assigned this Oct 25, 2024
@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Oct 25, 2024

/cc @manovotn (scheduler)

@manovotn
Copy link
Contributor

I think that all methods (except for isRunning()) should throw an UnsupportedOperationException.

+1, that makes sense to me as well

mkouba added a commit to mkouba/quarkus that referenced this issue Oct 31, 2024
@mkouba
Scheduler: throw UnsupportedOperationException if appropriate
08d72d4 
- also refactor the implementation and introduce BaseScheduler
- resolves quarkusio#44101
@mkouba mkouba mentioned this issue Oct 31, 2024
Merged
mkouba added a commit to mkouba/quarkus that referenced this issue Nov 4, 2024
@mkouba
Scheduler: throw UnsupportedOperationException if appropriate
e94ff3d 
- also refactor the implementation and introduce BaseScheduler
- resolves quarkusio#44101
@quarkus-bot quarkus-bot bot added this to the 3.17 - main milestone Nov 4, 2024
bschuhmann pushed a commit to bschuhmann/quarkus that referenced this issue Nov 16, 2024
@mkouba @bschuhmann
Scheduler: throw UnsupportedOperationException if appropriate
5355afb 
- also refactor the implementation and introduce BaseScheduler
- resolves quarkusio#44101
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mkouba mkouba

Labels
area/scheduler kind/enhancement New feature or request
Projects
None yet
Milestone
3.17.0.CR1
Development

Successfully merging a pull request may close this issue.

Scheduler: throw UnsupportedOperationException if appropriate mkouba/quarkus
2 participants
@mkouba @manovotn