-
Notifications
You must be signed in to change notification settings - Fork 9.2k
YARN-9532. SLSRunner.run() throws a YarnException when it fails to cr… #3709
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
Conversation
…eate the output directory
|
💔 -1 overall
This message was automatically generated. |
| System.err.println("ERROR: Cannot create output directory " | ||
| + outputFile.getAbsolutePath()); | ||
| throw new YarnException("Cannot create output directory"); | ||
| throw new IOException("Cannot create output directory"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any test or something to make sure we do not get a regression?
|
💔 -1 overall
This message was automatically generated. |
|
We're closing this stale PR because it has been open for 100 days with no activity. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. |
…eate the output directory
Description of PR
How was this patch tested?
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?