diff options
author | Reynold Xin <rxin@databricks.com> | 2015-05-30 23:36:32 -0700 |
---|---|---|
committer | Reynold Xin <rxin@databricks.com> | 2015-05-30 23:36:32 -0700 |
commit | 084fef76e90116c6465cd6fad7c0197c3e4d4313 (patch) | |
tree | cfb218424d0446172be4ddecdf9e9b30e8a16693 | |
parent | f7fe9e474417a68635a5ed1aa819d81a9be40895 (diff) | |
download | spark-084fef76e90116c6465cd6fad7c0197c3e4d4313.tar.gz spark-084fef76e90116c6465cd6fad7c0197c3e4d4313.tar.bz2 spark-084fef76e90116c6465cd6fad7c0197c3e4d4313.zip |
[SPARK-7976] Add style checker to disallow overriding finalize.
Author: Reynold Xin <rxin@databricks.com>
Closes #6528 from rxin/style-finalizer and squashes the following commits:
a2211ca [Reynold Xin] [SPARK-7976] Enable NoFinalizeChecker.
-rw-r--r-- | core/src/main/scala/org/apache/spark/api/python/PythonRDD.scala | 2 | ||||
-rw-r--r-- | scalastyle-config.xml | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/core/src/main/scala/org/apache/spark/api/python/PythonRDD.scala b/core/src/main/scala/org/apache/spark/api/python/PythonRDD.scala index a77bf42ce1..51388f01a3 100644 --- a/core/src/main/scala/org/apache/spark/api/python/PythonRDD.scala +++ b/core/src/main/scala/org/apache/spark/api/python/PythonRDD.scala @@ -843,6 +843,7 @@ private class PythonAccumulatorParam(@transient serverHost: String, serverPort: * An Wrapper for Python Broadcast, which is written into disk by Python. It also will * write the data into disk after deserialization, then Python can read it from disks. */ +// scalastyle:off no.finalize private[spark] class PythonBroadcast(@transient var path: String) extends Serializable { /** @@ -884,3 +885,4 @@ private[spark] class PythonBroadcast(@transient var path: String) extends Serial } } } +// scalastyle:on no.finalize diff --git a/scalastyle-config.xml b/scalastyle-config.xml index a0098169a0..072c48062c 100644 --- a/scalastyle-config.xml +++ b/scalastyle-config.xml @@ -96,7 +96,7 @@ <!-- <check level="error" class="org.scalastyle.scalariform.ReturnChecker" enabled="true"></check> --> <!-- <check level="error" class="org.scalastyle.scalariform.NullChecker" enabled="true"></check> --> <!-- <check level="error" class="org.scalastyle.scalariform.NoCloneChecker" enabled="true"></check> --> - <!-- <check level="error" class="org.scalastyle.scalariform.NoFinalizeChecker" enabled="true"></check> --> + <check level="error" class="org.scalastyle.scalariform.NoFinalizeChecker" enabled="true"></check> <!-- <check level="error" class="org.scalastyle.scalariform.CovariantEqualsChecker" enabled="true"></check> --> <!-- <check level="error" class="org.scalastyle.scalariform.StructuralTypeChecker" enabled="true"></check> --> <!-- <check level="error" class="org.scalastyle.file.RegexChecker" enabled="true"> --> |