GitHub user zhengruifeng opened a pull request:

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

    [Minor][DOC] Fix nits in JavaStreamingTestExample

    ## What changes were proposed in this pull request?
    
    Fix some nits discussed in 
https://github.com/apache/spark/pull/11776#issuecomment-198207419
    use !rdd.isEmpty instead of rdd.count > 0
    use static instead of AtomicInteger
    remove unneeded "throws Exception" 
    
    ## How was this patch tested?
    
    manual tests


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

    $ git pull https://github.com/zhengruifeng/spark je_fix

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

    https://github.com/apache/spark/pull/11821.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 #11821
    
----
commit dcce428b66aa0ffd0c468302598b823f8f7f4854
Author: Zheng RuiFeng <ruife...@foxmail.com>
Date:   2016-03-18T11:04:56Z

    create je_fix

commit 09ad928e3f5efde847eae324b648bfed227c0f34
Author: Zheng RuiFeng <ruife...@foxmail.com>
Date:   2016-03-18T11:07:47Z

    del throws

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to