GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/11968
[SPARK-14167][MINOR] Remove redundant `returns` in Scala code.
## What changes were proposed in this pull request?
Spark Scala code takes advantage of `return` statement as a control flow
in many cases, but it does not mean **redundant** `return` statements are
needed.
This PR tries to remove redundant `return` statement in Scala code.
Also, this fixes newly added java-lint errors(unused-imports, line-lengsth).
## How was this patch tested?
Pass the Jenkins tests.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dongjoon-hyun/spark SPARK-14167
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/11968.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 #11968
----
commit 534b202e294ea1095f1c940f6c3af91a98d00145
Author: Dongjoon Hyun <dongj...@apache.org>
Date: 2016-03-25T21:21:49Z
[SPARK-14167][MINOR] Remove redundant `returns` in Scala code.
Spark Scala code takes advantage of `return` statement as a control flow
in many cases, but it does not mean redundant `return` statements are
needed.
This PR tries to remove redundant `return` statement in Scala code.
Also, this fixes newly added java-lint errors(unused-imports, line-lengsth).
----
---
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
