Skip to content

[SPARK-29525][SQL][TEST] Fix the associated location already exists in SQLQueryTestSuite#26181

Closed
wangyum wants to merge 1 commit into
apache:masterfrom
wangyum:TestError
Closed

[SPARK-29525][SQL][TEST] Fix the associated location already exists in SQLQueryTestSuite#26181
wangyum wants to merge 1 commit into
apache:masterfrom
wangyum:TestError

Conversation

@wangyum

@wangyum wangyum commented Oct 20, 2019

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This PR fix Fix the associated location already exists in SQLQueryTestSuite:

build/sbt "~sql/test-only *SQLQueryTestSuite -- -z postgreSQL/join.sql"
...
[info] - postgreSQL/join.sql *** FAILED *** (35 seconds, 420 milliseconds)
[info]   postgreSQL/join.sql
[info]   Expected "[]", but got "[org.apache.spark.sql.AnalysisException
[info]   Can not create the managed table('`default`.`tt3`'). The associated location('file:/root/spark/sql/core/spark-warehouse/org.apache.spark.sql.SQLQueryTestSuite/tt3') already exists.;]" Result did not match for query #108

Why are the changes needed?

Fix bug.

Does this PR introduce any user-facing change?

No

How was this patch tested?

N/A

@SparkQA

SparkQA commented Oct 20, 2019

Copy link
Copy Markdown

Test build #112340 has finished for PR 26181 at commit aee5cbd.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@wangyum

wangyum commented Oct 20, 2019

Copy link
Copy Markdown
Member Author

cc @dongjoon-hyun

@dongjoon-hyun dongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for pinging me, @wangyum .
+1, LGTM. Merged to master.

Since the original PostgreSQL test also create temp table and this file is join.sql, this update is more accurate and better.

@wangyum wangyum deleted the TestError branch October 20, 2019 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants