[GitHub] [incubator-druid] gianm commented on issue #7595: Optimize overshadowed segments computation

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

gianm commented on issue #7595: Optimize overshadowed segments computation
URL: https://github.com/apache/incubator-druid/pull/7595#issuecomment-494654440
 
 
   > For the concern about DruidCoordinatorRuleRunner seeing a stale view, do you foresee issues with that acting on an older view of the overshadowed segments?
   
   It's not just a view of overshadowed segments, right? The entire timeline would be an older snapshot. I think as long as it's not 'surprisingly rolled back' then it is OK. Within any given coordinator leadership epoch, the snapshot at time T will be newer than the snapshot at previous times < T.
   
   So avoiding surprising rollbacks really boils down to avoiding a situation where the _new leader_ uses an older snapshot than the _old leader_. I think the best way to avoid this is to make sure that any snapshot used in a given leadership epoch must have been taken after the start of that epoch. (If it was from prior to the start of the epoch, it might be out of date and using it might lead to a surprising rollback.)

----------------------------------------------------------------
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


With regards,
Apache Git Services

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