Skip to content

Commit d22e12e

Browse files
gatorsmileRobert Kruszewski
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 <[email protected]> Closes apache#20969 from gatorsmile/setDefault.
1 parent 150b653 commit d22e12e

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)