Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20227#discussion_r161981394
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/AlreadyExistException.scala
---
@@ -31,7 +31,7 @@ class TableAlreadyExistsException(db: String, table:
String)
extends AnalysisException(s"Table or view '$table' already exists in
database '$db'")
class TempTableAlreadyExistsException(table: String)
--- End diff --
I think we should rename it. But @gatorsmile said "We do not want to
introduce a new exception type. In contrast, we planned to remove all these
exception sub-types because PySpark might output a confusing error message.".
So I revert it.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
