paul-rogers commented on PR #12852: URL: https://github.com/apache/druid/pull/12852#issuecomment-1204215869
We also had an offline discussion about `MultiPhaseParallelIndexingRowStatsTest`, which failed in one of my PRs. I see that the base class, `AbstractMultiPhaseParallelIndexingTest`, is adjusted in this PR, but `MultiPhaseParallelIndexingRowStatsTest` is not changed. Just want to confirm that this PR will fix the issue with that test: that we are comparing the time-based stats against expected values, and that the test randomly fails as a result since we can't guarantee execution duration. -- 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]
