-
Notifications
You must be signed in to change notification settings - Fork 9.1k
YARN-11830. Normalise path to fix file uri in windows #7756
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
💔 -1 overall
This message was automatically generated. |
Thanks for the fix @ASHISH-RANJAN59. Could you please run this PR using the local CI, zip and post the results. |
Adding output file for resourceManager tests run through local CI. None of the test failed with URISyntaxException |
@ASHISH-RANJAN59 I'm unable to download the logs. The download link points to the PR itself. |
Sure @GauthamBanasandra, adding the log file again below. |
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.
LGTM
Description of PR
UTs in yarn-resourcemanager is failing with below error
The URISyntaxException shows up in windows as file uri is different from linux system.
How was this patch tested?
The fix was tested by running UTs locally.
For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?