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

    https://github.com/apache/spark/pull/16664#discussion_r99520106
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
    @@ -1047,4 +1047,14 @@ object StaticSQLConf {
           "SQL configuration and the current database.")
         .booleanConf
         .createWithDefault(false)
    +
    +  val QUERY_EXECUTION_LISTENERS = 
buildConf("spark.sql.queryExecutionListeners")
    +    .doc("A comma-separated list of classes that implement 
QueryExecutionListener. When creating " +
    +      "a SparkSession, instances of these listeners will be added to it. 
These classes " +
    +      "needs to have a zero-argument constructor. If the specified class 
can't be found or" +
    +      " the class specified doesn't have a valid constructor the 
SparkSession creation " +
    +      "will fail with an exception.")
    +    .stringConf
    +    .toSequence
    +    .createWithDefault(Nil)
    --- End diff --
    
    Not sure whether we should make it internal or external. Let the others 
decide it. Either is fine to me.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to