Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/23031#discussion_r234314168
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/RuntimeConfig.scala
---
@@ -154,5 +154,9 @@ class RuntimeConfig private[sql](sqlConf: SQLConf = new
SQLConf) {
if (SQLConf.staticConfKeys.contains(key)) {
throw new AnalysisException(s"Cannot modify the value of a static
config: $key")
}
+ if (sqlConf.setCommandRejectsSparkConfs &&
+ ConfigEntry.findEntry(key) != null &&
!SQLConf.sqlConfEntries.containsKey(key)) {
--- End diff --
Actually, thinking of it, isn't `ConfigEntry.findEntry(key) != null`
redundant with the other check?
Removing that would also want by other settings that don't have constants
defined.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
