From d8c17025df163cbdb27e78974a5d6ed174fc04a0 Mon Sep 17 00:00:00 2001 From: Marco Pracucci Date: Thu, 21 Jan 2021 11:37:29 +0100 Subject: [PATCH] Fix TSDB head struct dump on querier error (#8379) * Fix TSDB head struct dump on querier error Signed-off-by: Marco Pracucci * Added mint/maxt to RangeHead.String() Signed-off-by: Marco Pracucci --- tsdb/head.go | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/tsdb/head.go b/tsdb/head.go index 53b0a0a3f..bf54511a8 100644 --- a/tsdb/head.go +++ b/tsdb/head.go @@ -1010,6 +1010,13 @@ func (h *RangeHead) Meta() BlockMeta { } } +// String returns an human readable representation of the range head. It's important to +// keep this function in order to avoid the struct dump when the head is stringified in +// errors or logs. +func (h *RangeHead) String() string { + return fmt.Sprintf("range head (mint: %d, maxt: %d)", h.MinTime(), h.MaxTime()) +} + // initAppender is a helper to initialize the time bounds of the head // upon the first sample it receives. type initAppender struct { @@ -1492,6 +1499,13 @@ func (h *Head) Close() error { return errs.Err() } +// String returns an human readable representation of the TSDB head. It's important to +// keep this function in order to avoid the struct dump when the head is stringified in +// errors or logs. +func (h *Head) String() string { + return "head" +} + type headChunkReader struct { head *Head mint, maxt int64