jayzhan211 commented on PR #11229:
URL: https://github.com/apache/datafusion/pull/11229#issuecomment-2219594670

   > the important part is `select(col("count(*)"))` which again seems to go 
directly to the logical planner builder and thus isn't an expression to be 
planned, or is that right?
   
   I agree with it. And, I'm not sure why the test looks like this and whether 
we should consider it valid or not. 
   
   IMO, `.select(vec![count(wildcard())])?` is the correct way to build the 
same plan.
   
   


-- 
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

Reply via email to