Skip to content

Commit

Permalink
[SPARK-31997][SQL][TESTS] Drop test_udtf table when SingleSessionSuit…
Browse files Browse the repository at this point in the history
…e test completed

### What changes were proposed in this pull request?
`SingleSessionSuite` not do `DROP TABLE IF EXISTS test_udtf` when test completed, then if we do mvn test `HiveThriftBinaryServerSuite`, the test case named `SPARK-11595 ADD JAR with input path having URL scheme` will FAILED because it want to re-create an exists table test_udtf.

### Why are the changes needed?
test suite shouldn't rely on their execution order

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
Manual test,mvn test SingleSessionSuite and HiveThriftBinaryServerSuite in order

Closes apache#28838 from LuciferYang/drop-test-table.

Authored-by: yangjie01 <yangjie01@baidu.com>
Signed-off-by: HyukjinKwon <gurwls223@apache.org>
  • Loading branch information
LuciferYang authored and HyukjinKwon committed Jun 16, 2020
1 parent f0e6d0e commit d24d27f
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -883,7 +883,7 @@ class SingleSessionSuite extends HiveThriftJdbcTest {
s"--conf ${HIVE_THRIFT_SERVER_SINGLESESSION.key}=true" :: Nil

test("share the temporary functions across JDBC connections") {
withMultipleConnectionJdbcStatement()(
withMultipleConnectionJdbcStatement("test_udtf")(
{ statement =>
val jarPath = "../hive/src/test/resources/TestUDTF.jar"
val jarURL = s"file://${System.getProperty("user.dir")}/$jarPath"
Expand Down

0 comments on commit d24d27f

Please sign in to comment.