[GitHub] [druid] maytasm commented on pull request #11025: Add an option for ingestion task to drop (mark unused) all existing segments that are contained by interval in the ingestionSpec

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org



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