Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Execute retention lease syncs under system context #53838

Commits on Mar 20, 2020

  1. Execute retention lease syncs under system context

    The retention lease syncs need to occur under the system context,
    because they are internal actions executed on behalf of the user. Today
    we are relying on this happening for background syncs by virtue of the
    fact that the context the syncs are created under is the system
    context. This is due to these occurring on the cluster state applier
    thread. However, there are situations where this does not hold such as
    when a timed out cluster state publication occurs, and the node where
    the shard is allocated is the elected master node. In that case, the
    context will be empty due to the fact that we do not reschedule
    publication under the system context. Currently, doing so runs us into
    some troubles with losing the existing context, possibly dropping
    deprecation headers. We could copy that context over when marking the
    current context as the system context, but the implications of that
    require some more investigation. For now, we explicitly mark the
    retention lease syncs as executing under the system context, as this is
    situation that we can reason about.
    jasontedor committed Mar 20, 2020
    Configuration menu
    Copy the full SHA
    7ac3edd View commit details
    Browse the repository at this point in the history