[SPARK-23242][SS][TESTS] Don't run tests in KafkaSourceSuiteBase twice

zsxwing · zsxwing · commit 073744985f43 · 2018-01-26T16:09:57.000-08:00
## What changes were proposed in this pull request? KafkaSourceSuiteBase should be abstract class, otherwise KafkaSourceSuiteBase will also run. ## How was this patch tested? Jenkins Author: Shixiong Zhu <zsxwing@gmail.com> Closes #20412 from zsxwing/SPARK-23242.
diff --git a/external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSourceSuite.scala b/external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSourceSuite.scala
@@ -546,7 +546,7 @@ class KafkaMicroBatchSourceSuite extends KafkaSourceSuiteBase {
   }
 }
 
-class KafkaSourceSuiteBase extends KafkaSourceTest {
+abstract class KafkaSourceSuiteBase extends KafkaSourceTest {
 
   import testImplicits._
 

Original file line number	Diff line number	Diff line change
`@@ -546,7 +546,7 @@ class KafkaMicroBatchSourceSuite extends KafkaSourceSuiteBase {`
`546`	`546`	`}`
`547`	`547`	`}`
`548`	`548`
`549`		`-class KafkaSourceSuiteBase extends KafkaSourceTest {`
	`549`	`+abstract class KafkaSourceSuiteBase extends KafkaSourceTest {`
`550`	`550`
`551`	`551`	`import testImplicits._`
`552`	`552`