sajjad-moradi opened a new pull request #7053:
URL: https://github.com/apache/incubator-pinot/pull/7053


   ## Description
    Currently AggregateMetric rule looks at selection columns and all of them 
have to be metric and inside a SUM function. It misses the fact that for 
group-by queries, groupby columns appear in selection with no function and 
that's a valid case for which AggergateMetrics can be turned on. This PR checks 
that scenario and acts accordingly.
   
   ## Testing Done
   - unit test
   - integration test


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

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



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

Reply via email to