Skip to content

Conversation

@peter-toth
Copy link
Contributor

@peter-toth peter-toth commented Jul 4, 2019

What changes were proposed in this pull request?

Before this PR inserting into a non-existing table returned a weird error message:

sql("INSERT INTO test VALUES (1)").show
org.apache.spark.sql.AnalysisException: unresolved operator 'InsertIntoTable 'UnresolvedRelation [test], false, false;;
'InsertIntoTable 'UnresolvedRelation [test], false, false
+- LocalRelation [col1#4]

after this PR the error message becomes:

org.apache.spark.sql.AnalysisException: Table not found: test;;
'InsertIntoTable 'UnresolvedRelation [test], false, false
+- LocalRelation [col1#0]

How was this patch tested?

Added a new UT.

@peter-toth
Copy link
Contributor Author

peter-toth commented Jul 4, 2019

@dongjoon-hyun this is a PR to fix the error message from here: #24860 (comment)

@dongjoon-hyun
Copy link
Member

Thank you, @peter-toth !

@SparkQA
Copy link

SparkQA commented Jul 4, 2019

Test build #107240 has finished for PR 25054 at commit 9ac59ba.

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

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM. Merged to master.
Thank you, @peter-toth !

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