[GitHub] [pinot] atris commented on pull request #7972: Broker Side validation for the query with aggregation and col but without group by

[GitBox <gi...@apache.org>]

atris commented on pull request #7972:
URL: https://github.com/apache/pinot/pull/7972#issuecomment-1005954218


   Just curious -- how much of a performance improvement do you see after this change? I am not saying we should not do it, but before we add a special case to the parser, it would be good to understand how much damage does the current approach do.


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org