Skip to content

Commit 16ef6ba

Browse files
gatorsmilecloud-fan
authored andcommitted
[SPARK-23826][TEST] TestHiveSparkSession should set default session
## What changes were proposed in this pull request? In TestHive, the base spark session does this in getOrCreate(), we emulate that behavior for tests. ## How was this patch tested? N/A Author: gatorsmile <gatorsmile@gmail.com> Closes #20969 from gatorsmile/setDefault.
1 parent 5cfd5fa commit 16ef6ba

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -159,10 +159,6 @@ private[hive] class TestHiveSparkSession(
159159
private val loadTestTables: Boolean)
160160
extends SparkSession(sc) with Logging { self =>
161161

162-
// TODO(SPARK-23826): TestHiveSparkSession should set default session the same way as
163-
// TestSparkSession, but doing this the same way breaks many tests in the package. We need
164-
// to investigate and find a different strategy.
165-
166162
def this(sc: SparkContext, loadTestTables: Boolean) {
167163
this(
168164
sc,

0 commit comments

Comments
 (0)