Skip to content
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

Interrupt in progress TSM compactions #8629

Merged
merged 1 commit into from
Jul 27, 2017
Merged

Interrupt in progress TSM compactions #8629

merged 1 commit into from
Jul 27, 2017

Conversation

jwilder
Copy link
Contributor

@jwilder jwilder commented Jul 25, 2017

When snapshots and compactions are disabled, the check to see if
the compaction should be aborted occurs in between writing to the
next TSM file. If a large compaction is running, it might take
a while for the file to be finished writing causing long delays.

This now interrupts compactions while iterating over the blocks to
write which allows them to abort immediately.

Required for all non-trivial PRs
  • Rebased/mergable
  • Tests pass
  • CHANGELOG.md updated
  • Sign CLA (if not already signed)

Fixes the TSM compactions part of #8614

@jwilder jwilder added this to the 1.4.0 milestone Jul 25, 2017
@jwilder jwilder requested a review from benbjohnson July 25, 2017 19:39
@jwilder jwilder added the review label Jul 25, 2017
@jwilder jwilder force-pushed the jw-compaction-abort branch 2 times, most recently from a014813 to 105c60a Compare July 25, 2017 21:21
When snapshots and compactions are disabled, the check to see if
the compaction should be aborted occurs in between writing to the
next TSM file.  If a large compaction is running, it might take
a while for the file to be finished writing causing long delays.

This now interrupts compactions while iterating over the blocks to
write which allows them to abort immediately.
@jwilder jwilder force-pushed the jw-compaction-abort branch from 105c60a to 18a02d5 Compare July 27, 2017 21:59
@jwilder jwilder merged commit 6e6cc99 into master Jul 27, 2017
@jwilder jwilder deleted the jw-compaction-abort branch July 27, 2017 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants