findepi commented on PR #13756: URL: https://github.com/apache/datafusion/pull/13756#issuecomment-2548322562
> The idea to coerce fixed size list to list is to simplify the logic to handle both kinds of list. 100% agreed > Unless this leads to issue otherwise I think we should keep this aggressive coercion. it does, because the logic was too eager (recursive where only single-step is needed). As proven by the unit test attached to the issue. I am naturally biased towards merging this PR, as it solves a real-life problem I encountered and had to workaround. @alamb @jayzhan211 what problem are we solving by not merging it? -- 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
