GitHub user bersprockets opened a pull request:

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

    [SPARK-22940][SQL] HiveExternalCatalogVersionsSuite should succeed on 
platforms that don't have wget

    ## What changes were proposed in this pull request?
    
    Modified HiveExternalCatalogVersionsSuite.scala to use Utils.doFetchFile to 
download different versions of Spark binaries rather than launching wget as an 
external process.
    
    On platforms that don't have wget installed, this suite fails with an error.
    
    @cloud-fan : would you like to check this change?
    
    ## How was this patch tested?
    
    1) test-only of HiveExternalCatalogVersionsSuite on several platforms. 
Tested bad mirror, read timeout, and redirects.
    2) ./dev/run-tests
    


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

    $ git pull https://github.com/bersprockets/spark SPARK-22940-alt

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

    https://github.com/apache/spark/pull/20147.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 #20147
    
----
commit 5fe679f3271692729e8b9a2a9351725a80fcf8f6
Author: Bruce Robbins <bersprockets@...>
Date:   2018-01-03T22:32:30Z

    Initial commit

commit 92e82d484a30a171141d2cdd1f800254fe33ceaf
Author: Bruce Robbins <bersprockets@...>
Date:   2018-01-03T23:12:09Z

    Remove test log messages

commit c5e835eb317d0b6970daef0ffc1214576874e8c6
Author: Bruce Robbins <bersprockets@...>
Date:   2018-01-04T02:39:03Z

    The 2 new methods should be private

----


---

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

Reply via email to