-
- Downloads
[SPARK-14508][BUILD] Add a new ScalaStyle Rule `OmitBracesInCase`
## What changes were proposed in this pull request? According to the [Spark Code Style Guide](https://cwiki.apache.org/confluence/display/SPARK/Spark+Code+Style+Guide) and [Scala Style Guide](http://docs.scala-lang.org/style/control-structures.html#curlybraces), we had better enforce the following rule. ``` case: Always omit braces in case clauses. ``` This PR makes a new ScalaStyle rule, 'OmitBracesInCase', and enforces it to the code. ## How was this patch tested? Pass the Jenkins tests (including Scala style checking) Author: Dongjoon Hyun <dongjoon@apache.org> Closes #12280 from dongjoon-hyun/SPARK-14508.
Showing
- core/src/main/scala/org/apache/spark/SparkContext.scala 4 additions, 8 deletionscore/src/main/scala/org/apache/spark/SparkContext.scala
- core/src/main/scala/org/apache/spark/SparkEnv.scala 1 addition, 2 deletionscore/src/main/scala/org/apache/spark/SparkEnv.scala
- core/src/main/scala/org/apache/spark/api/python/PythonHadoopUtil.scala 1 addition, 2 deletions.../scala/org/apache/spark/api/python/PythonHadoopUtil.scala
- core/src/main/scala/org/apache/spark/api/python/PythonRDD.scala 1 addition, 1 deletion...rc/main/scala/org/apache/spark/api/python/PythonRDD.scala
- core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala 4 additions, 8 deletions.../main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
- core/src/main/scala/org/apache/spark/deploy/master/Master.scala 16 additions, 32 deletions...rc/main/scala/org/apache/spark/deploy/master/Master.scala
- core/src/main/scala/org/apache/spark/deploy/master/MasterArguments.scala 1 addition, 1 deletion...cala/org/apache/spark/deploy/master/MasterArguments.scala
- core/src/main/scala/org/apache/spark/deploy/master/ZooKeeperPersistenceEngine.scala 1 addition, 2 deletions...ache/spark/deploy/master/ZooKeeperPersistenceEngine.scala
- core/src/main/scala/org/apache/spark/deploy/mesos/MesosClusterDispatcherArguments.scala 1 addition, 2 deletions.../spark/deploy/mesos/MesosClusterDispatcherArguments.scala
- core/src/main/scala/org/apache/spark/deploy/worker/ExecutorRunner.scala 2 additions, 4 deletions...scala/org/apache/spark/deploy/worker/ExecutorRunner.scala
- core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala 4 additions, 8 deletions...rc/main/scala/org/apache/spark/deploy/worker/Worker.scala
- core/src/main/scala/org/apache/spark/deploy/worker/WorkerArguments.scala 1 addition, 2 deletions...cala/org/apache/spark/deploy/worker/WorkerArguments.scala
- core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala 1 addition, 2 deletions.../apache/spark/executor/CoarseGrainedExecutorBackend.scala
- core/src/main/scala/org/apache/spark/metrics/MetricsSystem.scala 1 addition, 2 deletions...c/main/scala/org/apache/spark/metrics/MetricsSystem.scala
- core/src/main/scala/org/apache/spark/partial/BoundedDouble.scala 1 addition, 2 deletions...c/main/scala/org/apache/spark/partial/BoundedDouble.scala
- core/src/main/scala/org/apache/spark/rdd/DoubleRDDFunctions.scala 2 additions, 2 deletions.../main/scala/org/apache/spark/rdd/DoubleRDDFunctions.scala
- core/src/main/scala/org/apache/spark/rdd/OrderedRDDFunctions.scala 1 addition, 2 deletions...main/scala/org/apache/spark/rdd/OrderedRDDFunctions.scala
- core/src/main/scala/org/apache/spark/rdd/ParallelCollectionRDD.scala 3 additions, 6 deletions...in/scala/org/apache/spark/rdd/ParallelCollectionRDD.scala
- core/src/main/scala/org/apache/spark/rdd/PartitionerAwareUnionRDD.scala 1 addition, 2 deletions...scala/org/apache/spark/rdd/PartitionerAwareUnionRDD.scala
- core/src/main/scala/org/apache/spark/scheduler/InputFormatInfo.scala 2 additions, 4 deletions...in/scala/org/apache/spark/scheduler/InputFormatInfo.scala
Loading
Please register or sign in to comment