GitHub user HyukjinKwon opened a pull request:

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

    [MINORuse collect instead of show

    ## What changes were proposed in this pull request?
    
    This PR replace an effective `show()` to `collect()` to make the output 
silent.
    
    **Before:**
    
    ```
    test_simple_udt_in_df (pyspark.sql.tests.SQLTests) ... +---+----------+
    |key|       val|
    +---+----------+
    |  0|[0.0, 0.0]|
    |  1|[1.0, 1.0]|
    |  2|[2.0, 2.0]|
    |  0|[3.0, 3.0]|
    |  1|[4.0, 4.0]|
    |  2|[5.0, 5.0]|
    |  0|[6.0, 6.0]|
    |  1|[7.0, 7.0]|
    |  2|[8.0, 8.0]|
    |  0|[9.0, 9.0]|
    +---+----------+
    ```
    
    **After:**
    
    ```
    test_simple_udt_in_df (pyspark.sql.tests.SQLTests) ... ok
    ```
    
    ## How was this patch tested?
    
    Manually tested.

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

    $ git pull https://github.com/HyukjinKwon/spark minor-udf-test

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

    https://github.com/apache/spark/pull/22479.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 #22479
    
----
commit ceef555e1233fa722dfd4b66cfd2fda00b279929
Author: hyukjinkwon <gurwls223@...>
Date:   2018-09-20T01:32:25Z

    use collect instead of show

----


---

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

Reply via email to