Skip to content

Commit 0326b69

Browse files
heary-caogatorsmile
authored andcommitted
[MINOR][SQL][TEST] no uncache table in joinsuite test
## What changes were proposed in this pull request? At present, in test("broadcasted hash outer join operator selection") case, set the testData2 to _CACHE TABLE_, but no _uncache table_ testData2. It can make people confused. In addition, in the joinsuite test cases, clear the cache table of work by SharedSQLContext _spark.sharedState.cacheManager.clearCache_ to do, so we do not need to uncache table let's fix it. thanks. ## How was this patch tested? Existing test cases. Author: caoxuewen <[email protected]> Closes apache#18914 from heary-cao/uncache_table.
1 parent 0fcde87 commit 0326b69

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,6 @@ class JoinSuite extends QueryTest with SharedSQLContext {
126126
("SELECT * FROM testData join testData2 ON key = a where key = 2",
127127
classOf[BroadcastHashJoinExec])
128128
).foreach(assertJoin)
129-
sql("UNCACHE TABLE testData")
130129
}
131130

132131
test("broadcasted hash outer join operator selection") {
@@ -141,7 +140,6 @@ class JoinSuite extends QueryTest with SharedSQLContext {
141140
("SELECT * FROM testData right join testData2 ON key = a and key = 2",
142141
classOf[BroadcastHashJoinExec])
143142
).foreach(assertJoin)
144-
sql("UNCACHE TABLE testData")
145143
}
146144

147145
test("multiple-key equi-join is hash-join") {
@@ -476,7 +474,6 @@ class JoinSuite extends QueryTest with SharedSQLContext {
476474
).foreach(assertJoin)
477475
}
478476

479-
sql("UNCACHE TABLE testData")
480477
}
481478

482479
test("cross join with broadcast") {
@@ -565,7 +562,6 @@ class JoinSuite extends QueryTest with SharedSQLContext {
565562
Row("2", 3, 2) :: Nil)
566563
}
567564

568-
sql("UNCACHE TABLE testData")
569565
}
570566

571567
test("left semi join") {

0 commit comments

Comments
 (0)