GitHub user HyukjinKwon opened a pull request:

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

    [SPARK-19221][PROJECT INFRA][R] Add winutils binaries to the path in 
AppVeyor tests for Hadoop libraries to call native codes properly

    ## What changes were proposed in this pull request?
    
    It seems Hadoop libraries need winutils binaries for native libraries in 
the path.
    
    It is not a problem in tests for now because we are only testing SparkR on 
Windows via AppVeyor but it can be a problem if we run Scala tests via AppVeyor 
as below:
    
    ```
     - SPARK-18220: read Hive orc table with varchar column *** FAILED *** (3 
seconds, 937 milliseconds)
       org.apache.spark.sql.execution.QueryExecutionException: FAILED: 
Execution Error, return code -101 from 
org.apache.hadoop.hive.ql.exec.mr.MapRedTask. 
org.apache.hadoop.io.nativeio.NativeIO$Windows.access0(Ljava/lang/String;I)Z
       at 
org.apache.spark.sql.hive.client.HiveClientImpl$$anonfun$runHive$1.apply(HiveClientImpl.scala:625)
       at 
org.apache.spark.sql.hive.client.HiveClientImpl$$anonfun$runHive$1.apply(HiveClientImpl.scala:609)
       at 
org.apache.spark.sql.hive.client.HiveClientImpl$$anonfun$withHiveState$1.apply(HiveClientImpl.scala:283)
       ...
    ```
    
    This PR proposes to add it to the `Path` for AppVeyor tests.
    
    ## How was this patch tested?
    
    Manually via AppVeyor.
    
    **Before**
    
    
https://ci.appveyor.com/project/spark-test/spark/build/572-windows-complete/job/c4vrysr5uvj2hgu7
    
    **After**
    
    
https://ci.appveyor.com/project/spark-test/spark/build/549-windows-complete/job/gc8a1pjua2bc4i8m
    


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

    $ git pull https://github.com/HyukjinKwon/spark set-path-appveyor

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

    https://github.com/apache/spark/pull/16584.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 #16584
    
----
commit 7a8163cd006f669371b5188e8b55fc12fe5375d3
Author: hyukjinkwon <[email protected]>
Date:   2017-01-13T16:10:03Z

    Add hadoop.dll in AppVeyor tests for Hadoop libraries to call native 
libraries properly

----


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to