ODIn/spark-instrumented-optimizer
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

[MINOR][SQL][TEST] no uncache table in joinsuite test

Browse source 
## 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 <cao.xuewen@zte.com.cn>

Closes #18914 from heary-cao/uncache_table.
This commit is contained in:
caoxuewen 2017-08-14 09:33:22 -07:00 committed by gatorsmile
parent 0fcde87aad
commit 0326b69c98
1 changed files with 0 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala
View file

@ -126,7 +126,6 @@ class JoinSuite extends QueryTest with SharedSQLContext {
("SELECT * FROM testData join testData2 ON key = a where key = 2",
classOf[BroadcastHashJoinExec])
).foreach(assertJoin)
sql("UNCACHE TABLE testData")
}
test("broadcasted hash outer join operator selection") {
@ -141,7 +140,6 @@ class JoinSuite extends QueryTest with SharedSQLContext {
("SELECT * FROM testData right join testData2 ON key = a and key = 2",
classOf[BroadcastHashJoinExec])
).foreach(assertJoin)
sql("UNCACHE TABLE testData")
}
test("multiple-key equi-join is hash-join") {
@ -476,7 +474,6 @@ class JoinSuite extends QueryTest with SharedSQLContext {
).foreach(assertJoin)
}
sql("UNCACHE TABLE testData")
}
test("cross join with broadcast") {
@ -565,7 +562,6 @@ class JoinSuite extends QueryTest with SharedSQLContext {
Row("2", 3, 2) :: Nil)
}
sql("UNCACHE TABLE testData")
}
test("left semi join") {