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 single yarn get application status failed #3069

Closed
wants to merge 4 commits into from
Closed

fix single yarn get application status failed #3069

wants to merge 4 commits into from

Conversation

Eights-Li
Copy link
Contributor

What is the purpose of the pull request

fix single yarn get application status failed
deploy dolphin scheduler in single yarn hadoop cluster,
exec a yarn job will lead npe when get the yarn app status

Brief change log

  • Modify HadoopUtils

Verify this pull request

This pull request is already covered by existing tests, such as:
dolphinscheduler/common/utils/HadoopUtilsTest.java

@sonarcloud
Copy link

sonarcloud bot commented Jun 28, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

61.5% 61.5% Coverage
0.0% 0.0% Duplication

@zixi0825
Copy link
Member

+1

@davidzollo
Copy link
Contributor

davidzollo commented Jun 30, 2020

good job, but I have submitted a PR #3013 to solve this problem , and it has been merged

@davidzollo
Copy link
Contributor

so this PR will be closed

@davidzollo davidzollo closed this Jun 30, 2020
@Eights-Li Eights-Li deleted the dev-yarn-conf branch November 25, 2020 15:14
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 this pull request may close these issues.

3 participants