Add note on why SparkSubmitSuite needs ResetSystemProperties

markhamstra · Dec 25, 2014 · 4f4031d · 4f4031d
1 parent 4742a5b
commit 4f4031d
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala b/core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala
@@ -27,6 +27,8 @@ import org.apache.spark.util.{ResetSystemProperties, Utils}
 import org.scalatest.FunSuite
 import org.scalatest.Matchers
 
+// Note: this suite mixes in ResetSystemProperties because SparkSubmit.main() sets a bunch
+// of properties that neeed to be cleared after tests.
 class SparkSubmitSuite extends FunSuite with Matchers with ResetSystemProperties {
   def beforeAll() {
     System.setProperty("spark.testing", "true")