gianm commented on pull request #11940: URL: https://github.com/apache/druid/pull/11940#issuecomment-973002342
This patch is getting nailed for test coverage in UnifiedIndexerAppenderatorsManager's LimitedPoolIndexMerger. In looking into this I've noticed that the class does need more tests, but also that a lot of IndexMerger interface methods are currently unused. Rather than add tests for all these unused methods, I'm looking into removing them. -- 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]
