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

Removing planing loop #3913

Merged
merged 3 commits into from
Oct 6, 2023
Merged

Removing planing loop #3913

merged 3 commits into from
Oct 6, 2023

Commits on Oct 5, 2023

  1. Refactoring of the control plane.

    The two internal actors are replaced by two internal states.
    
    Their execution is not concurrent anymore,
    but we the code and the flow is simplified and we avoid some race conditions.
    
    The control plane is not getting the update notify signal through the
    broker anymore.
    
    The Notify RPC is entirely removed.
    
    Most of the test in the indexer scheduler become Control plane tests.
    fulmicoton committed Oct 5, 2023
    Configuration menu
    Copy the full SHA
    1a9ad5a View commit details
    Browse the repository at this point in the history
  2. Removing RefreshPlan loop.

    The RefreshPlan loop is not necessary since after we moved to
    the control plane as a metastore proxy world, the control
    plan cannot miss any Index/Source event.
    fulmicoton committed Oct 5, 2023
    Configuration menu
    Copy the full SHA
    619e5ae View commit details
    Browse the repository at this point in the history

Commits on Oct 6, 2023

  1. Configuration menu
    Copy the full SHA
    8a3ed29 View commit details
    Browse the repository at this point in the history