mustafasrepo commented on PR #9435: URL: https://github.com/apache/arrow-datafusion/pull/9435#issuecomment-1980285153
> I think, plan change comes from `optimize_projections` logical plan rule. where `ScalarFunctionDefinition` enum changes from `ScalarFunctionDefinition::Builtin` variant to `ScalarFunctionDefinition::Name` variant (to be resolved in physical planning). I think somehow this triggers the plan change. But I am not sure about the mechanism. Writing this in case, it is helpful. I have dubugged this further. It seemed like a small fix. Hence I sent a commit to fix this change. With this change, I also added the test [in my previous comment](https://github.com/apache/arrow-datafusion/pull/9435#discussion_r1513955386) to make sure we have desired coverage. -- 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]
