Skip to content

Break DB.Compact and DB.CompactHead and DB.CompactBlocks#6820

Merged
bwplotka merged 1 commit intoprometheus:masterfrom
codesome:break-compact
Feb 17, 2020
Merged

Break DB.Compact and DB.CompactHead and DB.CompactBlocks#6820
bwplotka merged 1 commit intoprometheus:masterfrom
codesome:break-compact

Conversation

@codesome
Copy link
Copy Markdown
Member

This just modularises the components in the DB.Compact() while having no changes in the logic. This will help external projects like Cortex get access to individual methods (mainly CompactHead).

/cc @thorfour @pracucci

@codesome codesome changed the title Break DB.Compact and DB.CompactHead and DB.CompactBlocks [WIP] Break DB.Compact and DB.CompactHead and DB.CompactBlocks Feb 14, 2020
@thorfour
Copy link
Copy Markdown
Contributor

These changes look great! Thank you!

@codesome codesome changed the title [WIP] Break DB.Compact and DB.CompactHead and DB.CompactBlocks Break DB.Compact and DB.CompactHead and DB.CompactBlocks Feb 14, 2020
Copy link
Copy Markdown
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Copy Markdown
Contributor

@pracucci pracucci left a comment

Choose a reason for hiding this comment

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

Thanks @codesome !

@bwplotka bwplotka merged commit 88af973 into prometheus:master Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants