rxin commented 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-478817998 Sorry guys your arguments don't really hold. If we deprecate only a small number of expressions, why does it matter than for each of them we'd need to add a few lines? If we don't want to pollute SQLConf because it is getting long, we can create a new object called SQLConfDeprecated or something like that. A single global deprecation flag doesn't work. Imagine a library you use that uses one of those deprecated configs. Now you will never get any warning anywhere ...
---------------------------------------------------------------- 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]
