-
Notifications
You must be signed in to change notification settings - Fork 465
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
Support more than one store per node #7889
Comments
This hasn't been tested. cc @darinpp |
@rmloveland, @mwang1026, what's the state of rebalancing across multiple stores? We have a number of docs issues around this, but I'm not sure if/how we should update the docs. |
We're in the same place re: rebalancing. It's actually been a problem of late so I'm hesitant to provide that recommendation. |
Thanks, @mwang1026. Is work ongoing on this? Should we close this issue and reopen to reflect current projects? This ticket is from the 20.2 roadmap... |
@jseldess , yes, @lunevalex worked on this in 20.2: Alex's change resulted in our safely rebalancing between stores that already existed on a node. IIRC, it does not support adding stores and rebalancing to an existing node and was not going to address the root problem of storing more per node. Ultimately, we decided to not prioritize documenting this update because it was not sufficiently customer impacting. |
As noted in #7666 and elsewhere, CockroachDB does not balance replicas among multiple stores per node. This limits the usefulness of running with multiple stores.
There is work ongoing to remove this limitation in the 20.2 cycle. Therefore, we will need to update docs accordingly when that work gets done.
Estimated scope of work:
Update the
cockroach start
docs accordingly. In particular, there is a note there about rebalancing that can probably (?) be removed as a result of this work (and that note might actually close Document that we do not balance replicas among multiple stores per node #7666, linked below).Possibly update the recommended production settings on storage to note that you can "put more stores on it"
Research other places this info might be needed in the docs (?), and do something about it
Related issues:
The text was updated successfully, but these errors were encountered: