maytasm commented on pull request #11025: URL: https://github.com/apache/druid/pull/11025#issuecomment-812201239
> Hmm, it's important that the old segments not be marked unused until after the new segments are loaded. The reason is that when the old segments are marked unused, they'll be dropped from historicals on the next coordinator run. The new segments might not be loaded by then. It'll lead to a period of data unavailability. Does this PR handle that case? In a Coordinator run, the RunRules happen before the UnloadUnusedSegments. Hence, we should be loading the new segments before dropping the existing segments. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
