Jefffrey commented on PR #18127: URL: https://github.com/apache/datafusion/pull/18127#issuecomment-3415814982
> I was planning on doing the same simplification as in #17982 where `nvl2(x, y, z)` would get simplified to `case when x is not null then y else z end`. The rationale is to try to make the various lazy, conditional logic variants (`nvl`, `nvl2`, `coalesce`) all aliases of `CaseExpr` so that we only have to optimise it once. Sounds good, I'll close this PR and close the refactor issue as unplanned Also fyi @kosiew since I see you self-assigned #17983 -- 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]
