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

pkg/ingester: added sync period flags #1438

Merged
merged 1 commit into from
Dec 18, 2019
Merged

pkg/ingester: added sync period flags #1438

merged 1 commit into from
Dec 18, 2019

Conversation

pstibrany
Copy link
Member

@pstibrany pstibrany commented Dec 18, 2019

-ingester.sync-period can be used to tell ingesters to cut chunks early (before they are full) to synchronize ingesters so that they put the same data into the new chunks. If they do, they generate fully equal chunks and then they don't need to store them.

To avoid very small chunks, additional parameter is added: -ingester.sync-min-utilization. When set to eg. 0.5, chunk must be at least 50% full (utilization) in order to be cut early for sync purposes.

To avoid all chunks to be cut at the same time, series fingerprint is used as an offset to sync period. We need some value that all ingesters know about and can use, and although series fingerprints can be remapped on ingesters differently, they are good enough.

Based on Bryan Boreham's work in Cortex.

Signed-off-by: Peter Štibraný peter.stibrany@grafana.com

Special notes for your reviewer:

Checklist

  • Documentation added
  • Tests updated

SyncPeriod can be used to tell ingesters to cut chunks early
(before they are full) to synchronize ingesters so that they put
the same data into the new chunks. If they do, they generate
fully equal chunks and then they don't need to store them.

To avoid very small chunks, additional parameter is added:
-ingester.sync-min-utilization. When set to eg. 0.5, chunk must be
at least 50% full (utilization) in order to be cut early for sync purposes.

Signed-off-by: Peter Štibraný <peter.stibrany@grafana.com>
Copy link
Collaborator

@slim-bean slim-bean left a comment

Choose a reason for hiding this comment

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

LGTM!

@slim-bean slim-bean merged commit dca244f into grafana:master Dec 18, 2019
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.

None yet

2 participants