jamisonbennett commented on a change in pull request #23398: [SPARK-26493][SQL]
Allow multiple spark.sql.extensions
URL: https://github.com/apache/spark/pull/23398#discussion_r244523719
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/StaticSQLConf.scala
##########
@@ -99,9 +99,10 @@ object StaticSQLConf {
.createWithDefault(false)
val SPARK_SESSION_EXTENSIONS = buildStaticConf("spark.sql.extensions")
- .doc("Name of the class used to configure Spark Session extensions. The
class should " +
+ .doc("List of the class names used to configure Spark Session extensions.
The classes should " +
Review comment:
I updated the documentation as suggested with respect to the ordering. I
think the comment about "listeners" is related to code that is close in
proximity to the code for this pull request, but it is a different
configuration item. So I didn't update the `spark.sql.queryExecutionListeners`
documentation. If you think that documentation should be updated, let me know
if I should include it as a part of this pull request or as a separate pull
request.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
