timsaucer commented on PR #17289: URL: https://github.com/apache/datafusion/pull/17289#issuecomment-3239515533
I think the best way forward is to merge this in with the limited support for only processing a single listarray and only literal/scalar other values. Then we open a separate issue for how we want to expand to handle arrays in the other argument positions. My reasoning is that what we have here I believe gives some immediate value, especially for cases where we have single input functions. Then we can make a design decision about how to expand it to work with multi-argument functions that get a `ColumnarValue::Array` input. I *suspect* that the run end encoding is the way to go, but that likely means an epic about building out our functions to support REE. That is overall a good thing, IMO. -- 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
