Jackie-Jiang commented on pull request #8170:
URL: https://github.com/apache/pinot/pull/8170#issuecomment-1035226449


   @hristo-stripe There are multiple cases where pinot can give approximate 
results for group-by queries:
   - Within segment, `numGroupsLimitReached` means new groups are ignored, 
which can potentially cause very inaccurate results in certain cases such as 
the group-by column is sorted
   - Trimming groups based on the ORDER-BY clause, which can remove the tail 
groups, and should give very close approximation if not 100% accurate.
   
   We should notify the user about the first case because that can result in 
very inaccurate results. For the second case, since it usually gives very close 
approximation, we don't want to mix it with the first case. If necessary, we 
can add another flag for the second 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: commits-unsubscr...@pinot.apache.org

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