HyukjinKwon commented on a change in pull request #27057:
[SPARK-29930][SQL][FOLLOW-UP] Allow only default value to be set for removed
SQL configs
URL: https://github.com/apache/spark/pull/27057#discussion_r362353286
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -170,6 +170,16 @@ object SQLConf {
}
}
+ /**
+ * The map contains info about removed SQL configs. Keys are SQL config
names,
+ * map values are tuples of default config value and the version in which
+ * the SQL config was removed.
+ */
+ val removedSQLConfigs = Map[String, (String, String)](
Review comment:
Oh, yeah. I was thinking about having this mechanism in SQL configurations
too but couldn't find some time to fix.
There is an example to follow in `SparkConf`, for example, see
`deprecatedConfigs`.
Can you match how it looks like? Then it should be good to go.
----------------------------------------------------------------
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]
