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

Is there any way to monitoring missfired job? #111

Closed
joint-song opened this issue Feb 20, 2024 · 5 comments · Fixed by #129
Closed

Is there any way to monitoring missfired job? #111

joint-song opened this issue Feb 20, 2024 · 5 comments · Fixed by #129

Comments

@joint-song
Copy link

I found some jobs because of short OutdatedThreshold or insufficient WorkerLimit configuration were missfired. How can I find these situations as soon as possible?

@reugn
Copy link
Owner

reugn commented Feb 21, 2024

I understand that the problem is that when using a worker pool, sometimes jobs are waiting to be processed, and their next execution time will likely be expired next time. In this case, I would suggest extending the OutdatedThreshold duration. But I agree, there is a problem with visibility. There are several ways to approach this, but have you thought about what would work best for you?

@joint-song
Copy link
Author

Java-quartz has MisfireInstruction to preventing this kind of outdated cron jobs. It has three options: MISFIRE_INSTRUCTION_IGNORE_MISFIRE_POLICY, MISFIRE_INSTRUCTION_FIRE_ONCE_NOW and MISFIRE_INSTRUCTION_DO_NOTHING to be configured. I think this is necessary for some scenarios.

@reugn
Copy link
Owner

reugn commented Mar 23, 2024

@joint-song, a configuration property may be introduced in the upcoming release to address this requirement.

@reugn
Copy link
Owner

reugn commented Apr 12, 2024

@joint-song, PR #129 introduces a dedicated channel for managing misfired jobs. Providing a channel instead of a configuration flag gives the user more flexibility and control. Let me know if this aligns with your requirements.

@reugn reugn linked a pull request Apr 15, 2024 that will close this issue
@joint-song
Copy link
Author

Yes, this solved my problem well, thanks for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants