Skip to content

Commit f4f4b91

Browse files
committed
[SPARK-52039][CORE][SQL][TESTS] Remove unnecessary with PrivateMethodTester from ClosureCleanerSuite2 and InsertSuite
### What changes were proposed in this pull request? This pr aims to remove unnecessary `with PrivateMethodTester` from `ClosureCleanerSuite2` and `InsertSuite`, ### Why are the changes needed? Code cleanup. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? ``` build/sbt clean "core/testOnly org.apache.spark.util.ClosureCleanerSuite2" build/sbt clean "hive/testOnly org.apache.spark.sql.hive.InsertSuite" ``` ### Was this patch authored or co-authored using generative AI tooling? No Closes #50830 from LuciferYang/SPARK-52039. Authored-by: yangjie01 <[email protected]> Signed-off-by: yangjie01 <[email protected]>
1 parent d48a9a3 commit f4f4b91

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

core/src/test/scala/org/apache/spark/util/ClosureCleanerSuite2.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ package org.apache.spark.util
1919

2020
import java.io.NotSerializableException
2121

22-
import org.scalatest.{BeforeAndAfterAll, PrivateMethodTester}
22+
import org.scalatest.BeforeAndAfterAll
2323

2424
import org.apache.spark.{SparkContext, SparkException, SparkFunSuite}
2525
import org.apache.spark.serializer.SerializerInstance
@@ -28,7 +28,7 @@ import org.apache.spark.serializer.SerializerInstance
2828
* Another test suite for the closure cleaner that is finer-grained.
2929
* For tests involving end-to-end Spark jobs, see {{ClosureCleanerSuite}}.
3030
*/
31-
class ClosureCleanerSuite2 extends SparkFunSuite with BeforeAndAfterAll with PrivateMethodTester {
31+
class ClosureCleanerSuite2 extends SparkFunSuite with BeforeAndAfterAll {
3232

3333
// Start a SparkContext so that the closure serializer is accessible
3434
// We do not actually use this explicitly otherwise

sql/hive/src/test/scala/org/apache/spark/sql/hive/InsertSuite.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ import java.util.Locale
2222

2323
import com.google.common.io.Files
2424
import org.apache.hadoop.fs.Path
25-
import org.scalatest.{BeforeAndAfter, PrivateMethodTester}
25+
import org.scalatest.BeforeAndAfter
2626

2727
import org.apache.spark.SparkException
2828
import org.apache.spark.sql.{QueryTest, _}
@@ -39,7 +39,7 @@ case class TestData(key: Int, value: String)
3939
case class ThreeColumnTable(key: Int, value: String, key1: String)
4040

4141
class InsertSuite extends QueryTest with TestHiveSingleton with BeforeAndAfter
42-
with SQLTestUtils with PrivateMethodTester {
42+
with SQLTestUtils {
4343
import spark.implicits._
4444

4545
override lazy val testData = spark.sparkContext.parallelize(

0 commit comments

Comments
 (0)