Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19649#discussion_r149765194
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/events.scala
---
@@ -62,6 +62,16 @@ case class DropDatabasePreEvent(database: String)
extends DatabaseEvent
case class DropDatabaseEvent(database: String) extends DatabaseEvent
/**
+ * Event fired before a database is altered.
+ */
+case class AlterDatabasePreEvent(database: String) extends DatabaseEvent
--- End diff --
Hi, @jerryshao .
We are adding `AlterTableEvent` and `AlterDatabaseEvent`. Can we have a
more specific PR title instead of `Add more ExternalCatalogEvent`?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
