xiangfu0 commented on PR #19396: URL: https://github.com/apache/pinot/pull/19396#issuecomment-5805916311
Rebased and simplified on top of `master` `257a0e7df2` (which includes #19412); the updated PR head is `7ecc01c0fd`. The patch shrank from 38 to 27 files and from 2,379 to 1,957 changed lines, with the benchmark harness kept temporary and out of the PR. Fresh matched JMH results on JDK 25, 400k rows, two servers, local + gRPC fan-in: | Ordered window | Rebased master | PR | Mean change | |---|---:|---:|---:| | Global | 193.400 ± 9.810 ms/op (3 forks, 15 measurements) | 137.801 ± 14.177 ms/op | -28.7% | | Partitioned | 247.859 ± 27.999 ms/op (2 forks, 10 measurements) | 184.532 ± 31.146 ms/op | -25.5% | Both JMH-reported intervals are non-overlapping. Local validation also passed: 89 focused runtime tests, all 243 `QueryCompilationTest` cases, and planner/runtime Spotless, Checkstyle, and license gates. Fresh exact-head CI is now running after the force-with-lease update. -- 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]
