KarpagamKarthikeyan commented on PR #23774: URL: https://github.com/apache/datafusion/pull/23774#issuecomment-5038838389
cc @Jefffrey @andygrove — this is my first contribution, so CI is currently awaiting a committer's approval. Would really appreciate a trigger when you have a moment. This adds a Spark-compatible `hypot` function to `datafusion-spark`, following the existing `math` function conventions (`exact(Float64, Float64) -> Float64`, `invoke` rather than `simplify`), backed by Rust's overflow-safe `f64::hypot`. Covered by sqllogictest cases (triples, doubles, NULL propagation, array path, and overflow-safety). Closes #23770. Thanks! -- 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]
