julianhyde commented on PR #3940:
URL: https://github.com/apache/calcite/pull/3940#issuecomment-2319127679

   Sorry, I think you need to touch that other code. Otherwise it will be 
inconsistent with AVG and no one will know why.
   
   I don't think your PR is in a fit state to commit. There are no comments, 
the query is a single line 120 characters long.
   
   Also, you don't need to 'imagine' why `ensureType` was added. (Since you 
tagged me in the case, are you asking me to do that work?) Please do some 
research based on the code history, and add your findings to the jira case.


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

Reply via email to