-
-
Notifications
You must be signed in to change notification settings - Fork 548
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
[5.x] Don't overlap git commits #11327
Conversation
@jesseleite I rolled this out to production and got this error:
So I guess it tried 3 times, failed, then threw an exception? Suggestions on best way to handle? Like if the lock goes away on its own later those commits will be handled, but there's no guarantee that there's another content change, so I'd rather ensure these actually get committed. Let's discuss. |
Not worth it right now, will leave the issue open and switch to scheduled commits. |
Closes #11322