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

[Bug] [Master] The task has been killed might still be dispatched #15661

Closed
3 tasks done
ruanwenjun opened this issue Mar 2, 2024 · 0 comments · Fixed by #15662
Closed
3 tasks done

[Bug] [Master] The task has been killed might still be dispatched #15661

ruanwenjun opened this issue Mar 2, 2024 · 0 comments · Fixed by #15662
Assignees
@ruanwenjun
Labels
bug Something isn't working
Milestone
3.2.2

Comments

@ruanwenjun
Copy link
Member

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

If the task is in dispatch waiting queue, then we kill the task it might still be dispatched.

What you expected to happen

The killed task will not be dispatched.

How to reproduce

  1. Create a task with delay.
  2. Start the workflow
  3. Kill the workflow instance.

Anything else

No response

Version

dev

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@ruanwenjun ruanwenjun added bug Something isn't working Waiting for reply Waiting for reply and removed Waiting for reply Waiting for reply labels Mar 2, 2024
@ruanwenjun ruanwenjun self-assigned this Mar 2, 2024
@ruanwenjun ruanwenjun added this to the 3.2.2 milestone Mar 2, 2024
@ruanwenjun ruanwenjun mentioned this issue Mar 2, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ruanwenjun ruanwenjun

Labels
bug Something isn't working
Projects
None yet
Milestone
3.2.2
Development

Successfully merging a pull request may close this issue.

[Bug] Fix task might be dispatched even if it has been killed ruanwenjun/dolphinscheduler
1 participant
@ruanwenjun