Github user cloud-fan commented on the issue:

    https://github.com/apache/spark/pull/21122
  
    To add some more color, the current `ExternalCatalog` is an abstract class 
and can already be implemented outside of Spark. However the problem is the 
listeners. Everytime we want to listen to one more event, we need to break the 
API(`createDatabase` and `doCreateDatabase`). this is very bad for a stable 
interface.
    
    The main goal of this PR is to pull out the listener stuff from 
`ExternalCatalog`, and make `ExternalCatalog` a pure interface.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to