HyukjinKwon commented on a change in pull request #35122:
URL: https://github.com/apache/spark/pull/35122#discussion_r780003488
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/StaticSQLConf.scala
##########
@@ -140,7 +140,8 @@ object StaticSQLConf {
val QUERY_EXECUTION_LISTENERS =
buildStaticConf("spark.sql.queryExecutionListeners")
.doc("List of class names implementing QueryExecutionListener that will be
automatically " +
"added to newly created sessions. The classes should have either a
no-arg constructor, " +
- "or a constructor that expects a SparkConf argument.")
+ "or a constructor that expects a SparkConf argument. User can use
SQLConf.get() to get the " +
+ "corresponding SessionState's conf when initialize
QueryExecutionListener.")
Review comment:
Yeah but let's don't document the internal stuff. I think it's a pretty
corner case in any event.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
