petern48 commented on PR #20702: URL: https://github.com/apache/datafusion/pull/20702#issuecomment-4042177870
Appreciate the little bump to get this moving again. So sorry for the delay. I didn't mean to keep you waiting. I definitely agree with your suggestion. I was cramming for a deadline, and wanted to wait until I had more time to investigate something a bit further. I vaguely remember just copy-pasting the `IS NOT NULL` part from the optimized output, so I had a small suspicion that the `CASE` statement was rewritten incorrectly to `IS NOT NULL`. Though I just manually checked it again, and it seems to optimize properly to `IS NULL`. Maybe I just wrote the case statement incorrectly, too 🤷. Anyway, lesson learned. I need to slow down a bit nowadays 🤦 , haha. Thanks for bearing with me. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
