You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2021/11/18 16:00:48 UTC

[GitHub] [druid] gianm commented on pull request #11940: Use intermediate-persist IndexSpec during multiphase merge.

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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org