diff options
author | Reynold Xin <rxin@apache.org> | 2013-11-06 13:27:47 -0800 |
---|---|---|
committer | Reynold Xin <rxin@apache.org> | 2013-11-06 13:27:47 -0800 |
commit | aadeda5e7697a433c82879033e758fbc403680dc (patch) | |
tree | 3208ff06042e86f95de185524b6c94241af309ae | |
parent | 951024feeadcf73b50c3c80ec9e75c7e2214a7a4 (diff) | |
parent | a0c4565183b1dcfa816f280ddf9c60c4c117df06 (diff) | |
download | spark-aadeda5e7697a433c82879033e758fbc403680dc.tar.gz spark-aadeda5e7697a433c82879033e758fbc403680dc.tar.bz2 spark-aadeda5e7697a433c82879033e758fbc403680dc.zip |
Merge pull request #144 from liancheng/runjob-clean
Removed unused return value in SparkContext.runJob
Return type of this `runJob` version is `Unit`:
def runJob[T, U: ClassManifest](
rdd: RDD[T],
func: (TaskContext, Iterator[T]) => U,
partitions: Seq[Int],
allowLocal: Boolean,
resultHandler: (Int, U) => Unit) {
...
}
It's obviously unnecessary to "return" `result`.
-rw-r--r-- | core/src/main/scala/org/apache/spark/SparkContext.scala | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/core/src/main/scala/org/apache/spark/SparkContext.scala b/core/src/main/scala/org/apache/spark/SparkContext.scala index 880b49e8ef..2bfe7c7631 100644 --- a/core/src/main/scala/org/apache/spark/SparkContext.scala +++ b/core/src/main/scala/org/apache/spark/SparkContext.scala @@ -796,11 +796,10 @@ class SparkContext( val cleanedFunc = clean(func) logInfo("Starting job: " + callSite) val start = System.nanoTime - val result = dagScheduler.runJob(rdd, cleanedFunc, partitions, callSite, allowLocal, + dagScheduler.runJob(rdd, cleanedFunc, partitions, callSite, allowLocal, resultHandler, localProperties.get) logInfo("Job finished: " + callSite + ", took " + (System.nanoTime - start) / 1e9 + " s") rdd.doCheckpoint() - result } /** |