somandal commented on PR #8758: URL: https://github.com/apache/pinot/pull/8758#issuecomment-1134900187
> do we have some numbers on the impact on performance? @kishoreg just to clarify, the purpose of this PR is to do the following: - Improve the accuracy of group-by only queries by adding a new query rewriter. This query rewriter just converts group-by queries to include order-by. - The new query rewriter is disabled by default because of the potential performance penalty. Only customers who need the improved accuracy and are willing to take the performance hit should enable this feature. - The query performance for group-by only will be no worse than running the same query with order-by added in. We can get perf numbers that compare a given query with group-by only and with group-by + order-by and get back to you on the difference. Hope this clarifies this change better. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
