imback82 commented on a change in pull request #30079:
URL: https://github.com/apache/spark/pull/30079#discussion_r507261691



##########
File path: 
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/v2/jdbc/JDBCTableCatalogSuite.scala
##########
@@ -77,8 +77,8 @@ class JDBCTableCatalogSuite extends QueryTest with 
SharedSparkSession {
     sql("DROP TABLE h2.test.to_drop")
     checkAnswer(sql("SHOW TABLES IN h2.test"), Seq(Row("test", "people")))
     Seq(
-      "h2.test.not_existing_table" -> "Table test.not_existing_table not 
found",
-      "h2.bad_test.not_existing_table" -> "Table bad_test.not_existing_table 
not found"
+      "h2.test.not_existing_table" -> "Table h2.test.not_existing_table not 
found",
+      "h2.bad_test.not_existing_table" -> "Table 
h2.bad_test.not_existing_table not found"

Review comment:
       Now that table is resolved before `drop table` runs, when the table is 
not found, it will always print the original identifier.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to