Blizzara opened a new pull request, #716: URL: https://github.com/apache/datafusion-comet/pull/716
## Which issue does this PR close? N/A ## Rationale for this change Spark-compatible "Cast" is implemented as a physical expr in Comet, but other downstream projects might like to use it as a logical expr. To facilitate it, this makes the core logic invokable through pub fn spark_cast_array. ## What changes are included in this PR? Makes some `Cast` struct's functions static, and moves those plus a bunch existing of static functions out of Cast impl. Moves ColumnarValue handling logic out of Cast::evaluate into `spark_cast_array`, and makes that public. Exports spark_cast_array from spark-expr/lib Recommend reviewing with "ignore whitespace" ON. ## How are these changes tested? Existing tests -- 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
