goldmedal commented on issue #11639: URL: https://github.com/apache/datafusion/issues/11639#issuecomment-2251886871
> We were working on the related `COUNT(*)`. At least in that case the `*` is just intercepted as the argument and replaced with `1`, so I don't think there'd be a conflict between that work and changing the expansion behavior in the projection itself. Thanks, @tshauck. I agree there is no conflict between them. I'm curious, which PR did you do for this? There are duplicate behaviors for the expansion (in the `select all` case, we will expand the expression twice). I want to check if this behavior can be removed. If the expansion in the builder is required for another case, I will also need to modify it. If not, I think we can just remove it and address the TODO comment. -- 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
