GitHub user hustfxj opened a pull request:

    https://github.com/apache/spark/pull/16443

    Remove query string from jar url for executor

    …ntains any query strings,  the "localName = name.split("/").last" won't 
get the expected jar, then "val url = new File(SparkFiles.getRootDirectory(), 
localName).toURI.toURL" will get invalid url. The bug fix is the same as 
[SPARK-17855]


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/hustfxj/spark hustfxj

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/16443.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #16443
    
----
commit d7da7cd2d57ddc996d16453529af7066f9e49bd4
Author: xiaojian.fxj <xiaojian....@alibaba-inc.com>
Date:   2016-12-31T12:56:24Z

    spark.jars support jar url with http protocal. However, if the url contains 
any query strings,  the "localName = name.split("/").last" won't get the 
expected jar, then "val url = new File(SparkFiles.getRootDirectory(), 
localName).toURI.toURL" will get invalid url. The bug fix is the same as 
[SPARK-17855]

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to