kazantsev-maksim opened a new pull request, #1321: URL: https://github.com/apache/datafusion-comet/pull/1321
## Which issue does this PR close? Related to Epic: https://github.com/apache/datafusion-comet/issues/1042 array_compact: SELECT array_compact(array(1, 2, 3, null)) => array(1, 2, 3) DataFusion' s array_compact has same behavior with Spark 's array_compact function Spark: https://docs.databricks.com/en/sql/language-manual/functions/array_compact.html DataFusion: https://datafusion.apache.org/user-guide/sql/scalar_functions.html#array-remove-all ## Rationale for this change Defined under Epic: https://github.com/apache/datafusion-comet/issues/1042 ## What changes are included in this PR? planner.rs: Maps Spark 's arrays_compact function to DataFusion array_remove_all_udf physical expression from Spark physical expression expr.proto: arrays_compact message has been added, QueryPlanSerde.scala: arrays_compact pattern matching case has been added, CometExpressionSuite.scala: A new UT has been added for arrays_compact function. ## How are these changes tested? A new UT has been added. -- 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
