-
Notifications
You must be signed in to change notification settings - Fork 598
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
feat: config change to scale in/out #3284
Comments
Do we consider source scale in this step?
I believe this will need to split into two steps: pre-mutate and mutate
Luckily, this step is unnecessary thanks to our shared storage design.
Also need design, this is not well-supported for now. Maybe we will need an actor manager that manages all actors running in the background, and abort them (or wait for them to exit gracefully) when we need to stop them. Luckily currently drop actors are handled by stop mutation, you might find it useful. |
Also need to wait for checkpoint, this is worth mentioning. The concurrent checkpoint PR is a really large change -- there'll be multiple barriers flowing in the system. I believe you'll need to take this into account. |
Thanks for your suggestion! I'm designing this part, collaborating with Ting Sun. We will carry out a detailed doc later.
|
Duplicated with #3750. |
Introduce config change to support scaling in/out.
To implement config change, we introduce a new pair of barrier: Pause/Resume.
Pause barrier marks the end of the previous configurations
Resume barrier marks the start of the new configurations
The config change process contains three steps:
Steps to scale in/out for each fragment
List of tasks:
The text was updated successfully, but these errors were encountered: