-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Wrapper method to easily retry deadlock exceptions #38030
Conversation
fe00005
to
2ed3880
Compare
2ed3880
to
3da12f9
Compare
Signed-off-by: Julius Härtl <[email protected]>
3da12f9
to
050c6d5
Compare
CI failure unrelated |
Hello! Are you familiar with our documentation process already? Changes that affect administrators should be documented here: While I understand sometimes it's handy to merge fast to get your changes in, it would be great if next time you could try to add the documentation before merging. You can find more information on that here: If all your documentation efforts are done, please remove the label 'documentation' and check the checkbox in your opening note and I'll stop bugging you :) Many thanks in advance and thanks again for your work! |
Started as nextcloud/documentation#10514 but will extend that a bit with more details later |
Wrapper around atomic() to retry after a retryable exception occurred
Certain transactions might need to be retried. This is especially useful in highly concurrent requests where a deadlocks is thrown by the database without waiting for the lock to be freed (e.g. due to MySQL/MariaDB deadlock detection)