Github user wzhfy commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19649#discussion_r148925392
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
 ---
    @@ -147,7 +154,15 @@ abstract class ExternalCatalog
        * Note: If the underlying implementation does not support altering a 
certain field,
        * this becomes a no-op.
        */
    -  def alterTable(tableDefinition: CatalogTable): Unit
    +  final def alterTable(tableDefinition: CatalogTable): Unit = {
    --- End diff --
    
    This depends on which level of details we wanna collect in the event. Are 
there any guidelines or documentation of what events spark should monitor? 
    Besides, partition changes are missing, I think it's necessary to also 
monitor these changes.


---

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

Reply via email to