jayzhan211 commented on PR #13756: URL: https://github.com/apache/datafusion/pull/13756#issuecomment-2544423933
> > Given that we coerce fixed size list to list, the return type of `array_element(fixed size list)` makes sense to be list. > > in the unit test, we ask for `array_element(list(fixed size list))` and we expect the return type to be `fixed size list`. in the fix, we make so that `array_element(list(T))` always returns `T`. The idea to coerce fixed size list to list is to simplify the logic to handle both kinds of list. Unless this leads to issue otherwise I think we should keep this aggressive coercion. -- 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
