LuciferYang commented on code in PR #37177:
URL: https://github.com/apache/spark/pull/37177#discussion_r921803395
##########
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala:
##########
@@ -71,7 +71,16 @@ object JdbcUtils extends Logging with SQLConfHelper {
} finally {
statement.close()
}
- }.isSuccess
+ } match {
+ case Success(_) => true
+ // This may be a known `TableAlreadyExist` exception, which is normal
and proves that the
+ // table does not exist, but perhaps some other exception, which in fact
does exist, will
+ // cause an error when renewing the table later. We need to know about
the exception stack
Review Comment:
I think we should handle the third state.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
