GitHub user gengliangwang opened a pull request:

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

    [SPARK-24776][SQL]Improve Avro unit test

    ## What changes were proposed in this pull request?
    
    Improve Avro unit test:
    1. use QueryTest/SharedSQLContext/SQLTestUtils, instead of the duplicated 
test utils.
    2. replace deprecated methods
    
    This is a follow up PR for #21760, the PR passes pull request tests but 
failed in: 
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Compile/job/spark-master-compile-maven-hadoop-2.6/7842/
    
    This PR is to fix it.
    ## How was this patch tested?
    Unit test.
    Compile with different commands:
    
    ```
    ./build/mvn --force -DzincPort=3643 -DskipTests -Phadoop-2.6 
-Phive-thriftserver -Pkinesis-asl -Pspark-ganglia-lgpl -Pmesos -Pyarn  compile 
test-compile
    ./build/mvn --force -DzincPort=3643 -DskipTests -Phadoop-2.7 
-Phive-thriftserver -Pkinesis-asl -Pspark-ganglia-lgpl -Pmesos -Pyarn  compile 
test-compile
    ./build/mvn --force -DzincPort=3643 -DskipTests -Phadoop-3.1 
-Phive-thriftserver -Pkinesis-asl -Pspark-ganglia-lgpl -Pmesos -Pyarn  compile 
test-compile
    
    ```

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

    $ git pull https://github.com/gengliangwang/spark improve_avro_test

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

    https://github.com/apache/spark/pull/21768.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 #21768
    
----
commit 26b88ca201a70283528f289cdd2e1e216fce6e7a
Author: Gengliang Wang <gengliang.wang@...>
Date:   2018-07-13T11:41:56Z

    improve AvroSuite

commit 4a7e60b265688ab6a8225fc3eea2ff3ebe8aec37
Author: Gengliang Wang <gengliang.wang@...>
Date:   2018-07-14T09:02:21Z

    fix compilation error

----


---

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

Reply via email to