GitHub user felixcheung opened a pull request:

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

    [SPARK-18807][SPARKR] Should suppress output print for calls to JVM methods 
with void return values

    ## What changes were proposed in this pull request?
    
    Several SparkR API calling into JVM methods that have void return values 
are getting printed out, especially when running in a REPL or IDE.
    example:
    ```
    > setLogLevel("WARN")
    NULL
    ```
    We should fix this to make the result more clear.
    
    ## How was this patch tested?
    
    manuall - I didn't find a expect_*() method in testthat for this


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

    $ git pull https://github.com/felixcheung/spark rinvis

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

    https://github.com/apache/spark/pull/16237.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 #16237
    
----
commit 9d8efafbbdfc79f2c8649bc1e5c4a1d7eccb73d0
Author: Felix Cheung <[email protected]>
Date:   2016-12-09T19:38:43Z

    wrap calls with invisible

commit 75f56c858b02e5bc409c4a35686f56d7058985a9
Author: Felix Cheung <[email protected]>
Date:   2016-12-09T19:40:52Z

    add test

----


---
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