mikamikasuki opened a new pull request, #26047: URL: https://github.com/apache/datafusion/pull/26047
## Which issue does this PR close? - Closes #25947 ## Rationale for this change NVL and IFNULL could coerce DECIMAL values to Float64, losing precision, and DATE/TIMESTAMP values to Utf8. This differs from COALESCE and prevents type-specific operations such as timestamp arithmetic. ## What changes are included in this PR? - Use COALESCE's common-type resolution for NVL and IFNULL. - Preserve the existing Boolean result for all-NULL calls and Boolean-to-numeric fallback behavior. - Add SQL logic tests for decimal types and precision, date/timestamp types, and timestamp arithmetic. ## What is the testing strategy for this PR? - NVL SQL logic tests, including the existing IFNULL numeric-fallback case. - Full extended workspace tests passed. - cargo fmt --all -- --check - cargo clippy --all-targets --all-features -- -D warnings ## Are there any user-facing changes? NVL and IFNULL preserve decimal, date, and timestamp types instead of converting them to Float64 or Utf8. -- 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]
