fuwhu commented on a change in pull request #26356: [SPARK-29707][SQL] Add 
configuration 'spark.sql.maxDataSourceScanMetadataStringLength'
URL: https://github.com/apache/spark/pull/26356#discussion_r342856661
 
 

 ##########
 File path: 
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
 ##########
 @@ -1961,6 +1961,14 @@ object SQLConf {
     .intConf
     .createWithDefault(25)
 
+  val MAX_DATA_SOURCE_SCAN_METADATA_STRING_LENGTH =
+    buildConf("spark.sql.maxDataSourceScanMetadataStringLength")
+      .doc("Maximum string length for one metadata of `DataSourceScanExec`. 
This is used in " +
 
 Review comment:
   Yes, currently it is only used in DataSourceScanExec. I checked all the 
implementations of TreeNode.simpleString method, now, no other instance has 
metadata contained in simpleString.
   I just refined the configuration name and description to make it be able to 
cover bigger scope if needed.
   @HyukjinKwon @wangyum  WDYT?

----------------------------------------------------------------
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.
 
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]

Reply via email to