cloud-fan commented on a change in pull request #23383: [SPARK-23817][SQL]
Create file source V2 framework and migrate ORC read path
URL: https://github.com/apache/spark/pull/23383#discussion_r247394773
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -1419,6 +1419,12 @@ object SQLConf {
.timeConf(TimeUnit.MILLISECONDS)
.createWithDefault(100)
+ val DISABLED_V2_FILE_READS = buildConf("spark.sql.files.disabledV2Reads")
Review comment:
Personally I think it's better to have one config for all v2
sources(read/write are separated), including streaming and batch, like
`spark.sql.sources.disabledV2Reads`. But let's keep it consistent with the
current codebase first, and change it later.
So +1 for `spark.sql.sources.disabledV2BatchReaders` in this PR.
----------------------------------------------------------------
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]
