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

    https://github.com/apache/spark/pull/22674#discussion_r223729445
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/util/QueryExecutionListener.scala 
---
    @@ -75,95 +76,69 @@ trait QueryExecutionListener {
      */
     @Experimental
     @InterfaceStability.Evolving
    -class ExecutionListenerManager private extends Logging {
    +class ExecutionListenerManager private[sql](session: SparkSession, 
loadExtensions: Boolean)
    --- End diff --
    
    nit: we shall add param comments.


---

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

Reply via email to