-
Notifications
You must be signed in to change notification settings - Fork 459
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
describe parallel writes during an insert #2300
Comments
These are mostly accurate, but in the second and fourth cases, the fact that you've omitted the RPC responses creates the impression that the sync on node3 is responsible for sending the final OK to the client. I've updated the second diagram with responses:
|
@bdarnell Good suggestions. I make the changes to the others. @Amruta-Ranade I can provide an outline for the page. |
Completed the changes on the diagrams to show return, corrected typo on insert DML |
New in 2.0 label could be treated as the entire parallel write is new in 2.0. refactor the graph to clarify parallel write to followers is existing 1.x functionality. sync to disk at leaseholder in parallel with the followers is new in 2.0. |
@robert-s-lee, is this essential for 2.0 or can we plan this work for post-release? |
Having the doc in 2.0 would be beneficial. Two specific areas: 1) highlight the potential performance improvements possible as outlined in cockroachdb/cockroach#19229 2) gateway connecting to the leaseholder for the reading and writing. Would additional outline in addition to the sequence diagrams help? |
Closing in favor of Closing in favor of #3873. |
@Amruta-Ranade the sequence diagrams (and accompanying .mmd files using mermaid) show how an insert is parallelized in detail. @bdarnell please check for accuracy again :)
Below is 1.x behavior.
Below is 2.0 behavior.
The write to leaderholder/raft leader is parallel with the followers that should improve performance.
The text was updated successfully, but these errors were encountered: