findepi commented on PR #13756: URL: https://github.com/apache/datafusion/pull/13756#issuecomment-2542298811
> So this problem is quite strange. How is it working today (without this change) 🤔 i believe the bug -- if we agree this is a bug -- is compensated by other factors. For example, at early planning stage it's totally OK to change expression types. Later, such a change triggers schema change assertion. I found this bug in case where array_element was inserted into the plan as a result of ScalarUDFImpl::simplify. At this stage it's "loose typing" is no longer OK. @alamb @jayzhan211 can you please review the attached unit test? Does it look sound, ie should it pass? Does it pass for you without other changes from the PR? -- 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
