Re: [PR] feat: Add support for `explode_outer` function for arrays [datafusion]

["comphead (via GitHub)" <gi...@apache.org>]

comphead commented on PR #22100:
URL: https://github.com/apache/datafusion/pull/22100#issuecomment-5052372668

   Thanks @athlcode.
   Since the code in the core it would actually benefit the DF to have outer `unnest`. For example `unnest` can be represented as a built in function or via joins https://github.com/apache/datafusion/pull/13824
   
   I checked some of engines for `unnest_outer` as a built in function and the same functionality represented mostly via joins. However if we support `unnest` it would prob makes sense to support `unnest_outer` as a builtin function. 
   
   Lets move tests from `explode.slt` into `unnest.slt` referring it as `unnest_outer`


-- 
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-unsubscribe@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org