pepijnve commented on issue #17801: URL: https://github.com/apache/datafusion/issues/17801#issuecomment-3499252909
> This is pretty gross but it might be ok enough to buy us some time That is rather nasty indeed, but it would sort of work. One concern I have is that it will not survive serialisation roundtrips. In the meantime https://github.com/apache/datafusion/actions/runs/19146115817/job/54724040455?pr=17813 shows some interesting results. Even if the simplification step is fixed, improving the nullability reporting of `case` causes schema changes further down the line when the optimiser eliminates constant expressions. This can cause something that may have seemed to be nullable to become not nullable. I'm thinking this should probably be allowed since it's making the post conditions stricter. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
