hvanhovell 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_r244412747
##########
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:
Please document in what order rules from multiple extensions are executed,
how listeners are invoked and how functions are registered (last one wins?). Be
sure to cover what happens if you add duplicate listeners/rules/functions etc...
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
