Skip to content

Chunk the cleanup queries to make sure they don't time out#22269

Merged
DeepDiver1975 merged 1 commit intomasterfrom
issue-22243-avoid-deadlock-with-lots-of-entries-to-cleanup
Feb 10, 2016
Merged

Chunk the cleanup queries to make sure they don't time out#22269
DeepDiver1975 merged 1 commit intomasterfrom
issue-22243-avoid-deadlock-with-lots-of-entries-to-cleanup

Conversation

@nickvergessen
Copy link
Copy Markdown
Contributor

Fix #22243

@PVince81 @MorrisJobke

@nickvergessen nickvergessen added this to the 9.0-current milestone Feb 10, 2016
@mention-bot
Copy link
Copy Markdown

By analyzing the blame information on this pull request, we identified @PVince81, @MorrisJobke and @blizzz to be potential reviewers

@nickvergessen nickvergessen force-pushed the issue-22243-avoid-deadlock-with-lots-of-entries-to-cleanup branch from bedecf8 to 51e6a92 Compare February 10, 2016 11:56
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CHUNK which means a piece, not JUNK which means garbage 😉

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also see number 3 in http://www.urbandictionary.com/define.php?term=junk
You might want to refrain using that word here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Woops, the commit message was right

@nickvergessen nickvergessen force-pushed the issue-22243-avoid-deadlock-with-lots-of-entries-to-cleanup branch from 51e6a92 to 0ebb205 Compare February 10, 2016 12:04
@MorrisJobke
Copy link
Copy Markdown
Contributor

Tested and works 👍

@karlitschek
Copy link
Copy Markdown
Contributor

👍

DeepDiver1975 added a commit that referenced this pull request Feb 10, 2016
…th-lots-of-entries-to-cleanup

Chunk the cleanup queries to make sure they don't time out
@DeepDiver1975 DeepDiver1975 merged commit 6ffb83a into master Feb 10, 2016
@DeepDiver1975 DeepDiver1975 deleted the issue-22243-avoid-deadlock-with-lots-of-entries-to-cleanup branch February 10, 2016 16:26
@lock
Copy link
Copy Markdown

lock Bot commented Aug 7, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

files:cleanup run in deadlock for bigger installations

6 participants