xiangfu0 commented on PR #19711: URL: https://github.com/apache/pinot/pull/19711#issuecomment-6786092041
@gortiz could you re-review `e3310a62e77`, restacked on #19396? The sort bypass now belongs to the opted-in bounded merge plan, and the default retains its final sort. Server-enriched EXPLAIN handles a root merge and retains LIMIT/OFFSET. The #19954 fix preserves randomly placed final singleton workers and partitioned keyed window sorts. All [14 hosted checks](https://github.com/apache/pinot/pull/19711/checks?sha=e3310a62e770db0bddabb2ab5cb844516bda31cc) and 1,149 local tests pass. November 4 benchmarks on master `fd1d3f7b59` and child `7fbdc1327a5`, before this restack, showed 4–15% paired mailbox latency gains within the child. Matched master p50 comparisons ranged from 18% slower to 6.5% faster. Leaf streaming had no clear offline gain and forced ON regressed the consuming LIMIT 8000 case by 46.5% in paired measurements; AUTO retained the existing path. Mailbox merging stays opt-in. -- 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]
