Skip to content

Conversation

@njnu-seafish
Copy link
Contributor

Purpose of the pull request

close #17359

Brief change log

1.add -i if enable sudo
2.change the condition of throwing the ExitCodeException

Pull Request Notice

Pull Request Notice

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

@SbloodyS SbloodyS added this to the 3.3.1 milestone Jul 23, 2025
@SbloodyS SbloodyS added the bug Something isn't working label Jul 23, 2025
@njnu-seafish njnu-seafish requested a review from SbloodyS July 24, 2025 01:24
@njnu-seafish njnu-seafish requested a review from SbloodyS July 25, 2025 01:15
@sonarqubecloud
Copy link

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.

LGTM

@SbloodyS SbloodyS requested review from Gallardot and ruanwenjun July 25, 2025 05:39
@ruanwenjun ruanwenjun changed the title [Fix-17359][Task-API] fix the bug about killing yarn application fail [Fix-17359][Task-API] Use sudo -i to avoid killing yarn application failed. Jul 25, 2025
Copy link
Member

@ruanwenjun ruanwenjun left a comment

Choose a reason for hiding this comment

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

LGTM

@ruanwenjun ruanwenjun merged commit f065f96 into apache:dev Jul 25, 2025
70 checks passed
eco8848 pushed a commit to eco8848/dolphinscheduler that referenced this pull request Aug 8, 2025
davidzollo pushed a commit to davidzollo/dolphinscheduler that referenced this pull request Oct 27, 2025
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] [task api] kill yarn application failed

3 participants