kfaraz commented on PR #18622: URL: https://github.com/apache/druid/pull/18622#issuecomment-3402159177
> How are we going to track this? Is there a github issue that we could create sub issues for different groups of tests? I fear it will fall off once they are moved since people will be happy enough with how much nicer CI feels. I don't really think it warrants a github issue since the tests are perfectly functional in their current form as well. The refactor would be more of a code beautification than anything else. It would involve use of task builders and more of the `EmbeddedClusterApis`, `LatchableEmitter` and less reliance on `ITRetryUtil`. It doesn't really affect the quality of the tests or their coverage. I am going to add some comments in `AbstractIndexerTest` to this effect. Please let me know if you still feel that we should create a GitHub issue. -- 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]
