-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[SPARK-11595] [SQL] [BRANCH-1.5] Fixes ADD JAR when the input path contains URL scheme #9570
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
Test build #45372 has finished for PR 9570 at commit
|
Test build #2022 has finished for PR 9570 at commit
|
Is the failed test a legitimate one? |
Test build #45639 has finished for PR 9570 at commit
|
Let's add this test in thfitserver tests. |
val jarPath = TestHive.getHiveFile("TestUDTF.jar").getCanonicalPath | ||
|
||
// SPARK-11595 Fixes ADD JAR when input path contains URL scheme | ||
val jarURL = s"file://$jarPath" |
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.
Should we also check a URL without a scheme?
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 it possible to remove an added jar? I'm afraid if we add the other test case, these two may interfere with each other.
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.
Actually there are other test cases that already covered the case without a scheme. So I think it's fine.
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.
I think it is not possible.
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.
ok
Thanks! Merging to branch 1.5. |
…ains URL scheme This PR backports #9569 to branch-1.5. Author: Cheng Lian <lian@databricks.com> Closes #9570 from liancheng/spark-11595.for-branch-1.5.
I have merged it to branch 1.5. |
This PR backports #9569 to branch-1.5.