maropu commented on a change in pull request #32482:
URL: https://github.com/apache/spark/pull/32482#discussion_r628885975
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -1090,6 +1090,18 @@ object SQLConf {
.booleanConf
.createWithDefault(true)
+ val CACHE_DISABLE_CONFIGS =
+ buildConf("spark.sql.cache.disableConfigs")
+ .doc("Configurations needs to be turned off, to avoid regression for
cached query, so that " +
+ "the outputPartitioning of the underlying cached query plan can be
leveraged later.")
Review comment:
I think this comment is for developers, so it is difficult for a user to
understand it. Could you brush up it more?
--
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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
