yashmayya opened a new pull request, #13514:
URL: https://github.com/apache/pinot/pull/13514

   - The grouping algorithm and group limit configurations are described here - 
https://docs.pinot.apache.org/users/user-guide-query/query-syntax/grouping-algorithm
   - Currently, if the trim size is larger than half the trim threshold, it is 
overridden to half the trim threshold in order to avoid excessive trimming.
   - However, this can lead to inaccurate results if the trim size has 
intentionally been configured to a high value based on the data pattern.
   - The better thing to do in this case would be to double the trim threshold 
instead, which can lead to higher memory usage but will respect the desired 
accuracy level.


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