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

[Fix-16467][dolphinscheduler-master] Can't stop the subProcess instance successfully via STOP its master process instance #16470

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

topsli
Copy link
Contributor

@topsli topsli commented Aug 15, 2024

Purpose of the pull request

fix #16467

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

…ce successfully via STOP its master process instance
Copy link

boring-cyborg bot commented Aug 15, 2024

Thanks for opening this pull request! Please check out our contributing guidelines. (https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/join/pull-request.md)

@SbloodyS SbloodyS added bug Something isn't working first time contributor First-time contributor labels Aug 15, 2024
@SbloodyS SbloodyS added this to the 3.3.0 milestone Aug 15, 2024
Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So far we are working on refactoring this kind of logic in #16423 . This PR is temporarily suspended until the refactoring is complete. @topsli

Copy link

sonarcloud bot commented Aug 15, 2024

@github-actions github-actions bot added the UI ui and front end related label Aug 16, 2024
@github-actions github-actions bot removed the UI ui and front end related label Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend bug Something isn't working first time contributor First-time contributor
Projects
None yet
2 participants