-
- Downloads
Merge pull request #490 from hsaputra/modify_checkoption_with_isdefined
Replace the check for None Option with isDefined and isEmpty in Scala code Propose to replace the Scala check for Option "!= None" with Option.isDefined and "=== None" with Option.isEmpty. I think this, using method call if possible then operator function plus argument, will make the Scala code easier to read and understand. Pass compile and tests.
No related branches found
No related tags found
Showing
- core/src/main/scala/org/apache/spark/Partitioner.scala 1 addition, 1 deletioncore/src/main/scala/org/apache/spark/Partitioner.scala
- core/src/main/scala/org/apache/spark/deploy/master/ui/ApplicationPage.scala 1 addition, 1 deletion...a/org/apache/spark/deploy/master/ui/ApplicationPage.scala
- core/src/main/scala/org/apache/spark/metrics/MetricsConfig.scala 1 addition, 1 deletion...c/main/scala/org/apache/spark/metrics/MetricsConfig.scala
- core/src/main/scala/org/apache/spark/partial/ApproximateActionListener.scala 1 addition, 1 deletion.../org/apache/spark/partial/ApproximateActionListener.scala
- core/src/main/scala/org/apache/spark/partial/PartialResult.scala 8 additions, 8 deletions...c/main/scala/org/apache/spark/partial/PartialResult.scala
- core/src/main/scala/org/apache/spark/rdd/RDD.scala 1 addition, 1 deletioncore/src/main/scala/org/apache/spark/rdd/RDD.scala
- core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala 1 addition, 1 deletion.../main/scala/org/apache/spark/scheduler/DAGScheduler.scala
- core/src/main/scala/org/apache/spark/scheduler/Stage.scala 1 addition, 1 deletioncore/src/main/scala/org/apache/spark/scheduler/Stage.scala
- core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala 2 additions, 2 deletions.../scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala
- core/src/main/scala/org/apache/spark/storage/BlockManagerWorker.scala 1 addition, 1 deletion...n/scala/org/apache/spark/storage/BlockManagerWorker.scala
- core/src/main/scala/org/apache/spark/storage/MemoryStore.scala 1 addition, 1 deletion...src/main/scala/org/apache/spark/storage/MemoryStore.scala
- core/src/main/scala/org/apache/spark/util/Utils.scala 1 addition, 1 deletioncore/src/main/scala/org/apache/spark/util/Utils.scala
- core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala 1 addition, 1 deletion...cala/org/apache/spark/scheduler/TaskSetManagerSuite.scala
- core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala 63 additions, 63 deletions...st/scala/org/apache/spark/storage/BlockManagerSuite.scala
Loading
Please register or sign in to comment