HyukjinKwon edited a comment on issue #24259: [SPARK-27328][SQL] Add 'deprecated' in ExpressionDescription for extended usage and SQL doc URL: https://github.com/apache/spark/pull/24259#issuecomment-478808428 It is a few pieces of codes, yes. I was worrying about we start to have many flags under `SQLConf` (where we already have a lot). From what I have seen, it always grows but never decreases. Adding a configuration for each deprecation with throwing an exception looked like it makes harder to deprecate and get out of the legacy stuff. (for instance, the SparkR and PySpark tests had to be fixed few times in #24195).
---------------------------------------------------------------- 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]
