-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Suggest to support timeout for JobConfiguration #1464
Comments
It is a necessary feature, I just set the label to new feature. |
I can make it, Could you assign to me. @terrymanu @TeslaCN |
Hi @skaic |
Thanks @TeslaCN . It is a necessary feature for me , I have a idea and give me some time to write, Thanks. |
Core: We start a process in new thread and waiting it。 When time out, call the interrupt() of the thread And cloes() of the thread socket 。 So, Ours Job can try to find a timeout By Check isInterrupted ( 代码如下:
Job:
|
of course , the waiting timeout code can make it better . |
I don't think it is elegant that let user write the |
Right, But Java deprecated We just providing a solution to stop |
Job execution timeout for quartz is doing like that. see it https://www.quartz-scheduler.net/documentation/quartz-3.x/packages/quartz-plugins.html#:~:text=%23-,JobInterruptMonitorPlugin,-This%20plugin%20catches |
JobConfiguration should add timeout for users. If the job executes timeout event if, in sharding level, we can trace it and monitor it further.
As for dag, we can decide to wait for the max time for this node.
The text was updated successfully, but these errors were encountered: