FrankChen021 opened a new pull request, #20305: URL: https://github.com/apache/druid/pull/20305
### Description This PR reduces the local runtime of `MergingRowIteratorTest` from approximately 24 seconds to 5 seconds while retaining systematic coverage of merge ordering and mark semantics. The exhaustive 5-element sequence test now verifies merge order once for each sequence combination. Mark behavior is tested separately across all possible 4-element sequence combinations and every output mark position. This reduces mark-verification combinations from 2,699,004 to 341,376 (87% fewer), while still exercising different heap layouts, equal and changing timestamps, and iterator exhaustion. Assertion failure messages are supplied lazily so expensive diagnostics are constructed only when an assertion fails. ### Benchmark | Version | Run 1 | Run 2 | Average | | --- | ---: | ---: | ---: | | Current `master` | 23.942s | 23.941s | 23.942s | | Revised test | 4.890s | 5.356s | 5.123s | This is approximately a 78.6% reduction in test runtime. ### Verification ``` mvn test -pl processing -Dtest=org.apache.druid.segment.MergingRowIteratorTest -Pskip-static-checks -Dweb.console.skip=true -T1C ``` All 7 tests pass inTokens. -- 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]
