blaginin commented on PR #13576: URL: https://github.com/apache/datafusion/pull/13576#issuecomment-2539236344
Feels like the right move is to add a warning to that option, see if we get any requests or issues, and if not, just remove it completely. I also don’t think this option should exist. The value is either case-sensitive or it’s not - we know that for sure for each value. If we’re missing some edge case, we should fix it instead of normalizing the value. Normalizing might solve one problem but could also break other stuff, like like integration. If you’re good with that, @ozankabak @berkaysynnada, I’ll deprecate the option -- 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. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
