Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upUpon append of a new series, should watermarks be written immediately? #309
Comments
ghost
assigned
matttproud
Jun 20, 2013
This comment has been minimized.
This comment has been minimized.
|
Right now we have to wait for the curator to run until we can query a completely new series :-/ |
This comment has been minimized.
This comment has been minimized.
|
I would suggest checking whether in-memory series for the given fingerprint is empty before deeming the fingerprint "too old". This would be best, as the watermarks ideally should reflect disk state. |
This comment has been minimized.
This comment has been minimized.
|
To narrow the error possibility down more: we can query series which a) have been curated before (have a disk watermark), or b) have received a memory sample already during the current Prometheus run. The edge case are timeseries which have not been curated and which haven't been seen yet since the last restart. This can happen if you briefly run a new Prometheus, then restart it. Suddenly you are not able to query the samples from the first run anymore (until you receive more of the same timeseries). |
This comment has been minimized.
This comment has been minimized.
|
Done in a1a23fb We also found out that the curator actually never writes watermarks for series that don't have watermarks already. So did we never actually have watermarks before? |
juliusv
closed this
Jun 28, 2013
simonpasquier
pushed a commit
to simonpasquier/prometheus
that referenced
this issue
Oct 12, 2017
This comment has been minimized.
This comment has been minimized.
lock
bot
commented
Mar 25, 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. |
juliusv commentedJun 20, 2013
See my new comment on already merged earlier PR: https://github.com/prometheus/prometheus/pull/291/files#r4810106