1ffa627ffb
### What changes were proposed in this pull request? 1. Put all deprecated SQL configs the map `SQLConf.deprecatedSQLConfigs` with extra info about when configs were deprecated and additional comments that explain why a config was deprecated, what an user can use instead of it. Here is the list of already deprecated configs: - spark.sql.hive.verifyPartitionPath - spark.sql.execution.pandas.respectSessionTimeZone - spark.sql.legacy.execution.pandas.groupedMap.assignColumnsByName - spark.sql.parquet.int64AsTimestampMillis - spark.sql.variable.substitute.depth - spark.sql.execution.arrow.enabled - spark.sql.execution.arrow.fallback.enabled 2. Output warning in `set()` and `unset()` about deprecated SQL configs ### Why are the changes needed? This should improve UX with Spark SQL and notify users about already deprecated SQL configs. ### Does this PR introduce any user-facing change? Yes, before: ``` spark-sql> set spark.sql.hive.verifyPartitionPath=true; spark.sql.hive.verifyPartitionPath true ``` After: ``` spark-sql> set spark.sql.hive.verifyPartitionPath=true; 20/01/03 21:28:17 WARN RuntimeConfig: The SQL config 'spark.sql.hive.verifyPartitionPath' has been deprecated in Spark v3.0.0 and may be removed in the future. This config is replaced by spark.files.ignoreMissingFiles. spark.sql.hive.verifyPartitionPath true ``` ### How was this patch tested? Add new test which registers new log appender and catches all logging to check that `set()` and `unset()` log any warning. Closes #27092 from MaxGekk/group-deprecated-sql-configs. Authored-by: Maxim Gekk <max.gekk@gmail.com> Signed-off-by: HyukjinKwon <gurwls223@apache.org> |
||
---|---|---|
.. | ||
benchmarks | ||
src | ||
v1.2/src | ||
v2.3/src | ||
pom.xml |