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
Try to prevent a deadlock when writing options #15666
Conversation
@@ -78,11 +78,12 @@ public function getAllRememberToInvalidateArchivedReportsLater() | |||
// we do not really have to get the value first. we could simply always try to call set() and it would update or | |||
// insert the record if needed but we do not want to lock the table (especially since there are still some | |||
// MyISAM installations) | |||
$values = Option::getLike($this->rememberArchivedReportIdStart . '%'); | |||
$values = Option::getLike('%' . $this->rememberArchivedReportIdStart . '%'); |
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.
Sometimes I see '%_' .
and other times '%' .
, could that be an issue?
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.
@diosmosis not really. I just copied behaviour what was done on the right part, to have the same on the left. It'll not make a difference though
I don't see a change to the |
@diosmosis yes there should! Will add it now. I used to have it there but must have been undone by some other reverts |
@diosmosis actually it works because it is already partially sent with the |
looks good to merge if tests pass 👍 |
refs #15545 (comment)
Invalidator tests passed for me locally. Not sure I missed something maybe?