beliefer 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_r244477766
##########
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:
A suggestion of update the comment is replace 'List of the class names used
to configure Spark Session extensions. The classes should implement
Function1[SparkSessionExtension, Unit], and must have a no-args constructor.'
to 'A comma-separated list of classes that implement
Function1[SparkSessionExtension, Unit] used to configure Spark Session
extensions. The classes must have a no-args constructor.'
----------------------------------------------------------------
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]
