-
Notifications
You must be signed in to change notification settings - Fork 131
[CDTOOL-1056] Implement new mechanism for serializing mutating requests. #715
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
Conversation
The new method ensures that mutating requests being issued by separate Client objects are serialized, which will resolve issues with applications which use multiple Client objects in goroutines. The default mode serializes all mutating requests, but the mechanism offers a way to group requests by a 'resource ID' to minimize the performance impact.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this will work if clients opt into it.
This should work even if clients don't opt-in to it, because there's a default 'resource ID' which will capture all mutating operations. Opting in is only necessary to improve performance by allowing mutating operations for separate resources to happen in parallel. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is a good solution! Thanks for doing this!
The new method ensures that mutating requests being issued by separate Client objects are serialized, which will resolve issues with applications which use multiple Client objects in goroutines.
The default mode serializes all mutating requests, but the mechanism offers a way to group requests by a 'resource ID' to minimize the performance impact.
All Submissions:
New Feature Submissions:
Changes to Core Features:
User Impact
Are there any considerations that need to be addressed for release?