Avoid expensive log.Valuer evaluation for disallowed levels (thanos-i…

…o#6322)

Signed-off-by: Xiaochao Dong (@damnever) <the.xcdong@gmail.com>
Signed-off-by: Giedrius Statkevičius <giedrius.statkevicius@vinted.com>

pkg/logging/logger.go

@@ -46,11 +46,14 @@ func NewLogger(logLevel, logFormat, debugName string) log.Logger {
 		logger = log.NewJSONLogger(log.NewSyncWriter(os.Stderr))
 	}
 
+	// Sort the logger chain to avoid expensive log.Valuer evaluation for disallowed level.
+	// Ref: https://github.com/go-kit/log/issues/14#issuecomment-945038252
+	logger = log.With(logger, "ts", log.DefaultTimestampUTC, "caller", log.Caller(5))
 	logger = level.NewFilter(logger, lvl)
 
 	if debugName != "" {
 		logger = log.With(logger, "name", debugName)
 	}
 
-	return log.With(logger, "ts", log.DefaultTimestampUTC, "caller", log.DefaultCaller)
+	return logger
 }

pkg/logging/logger_test.go

@@ -0,0 +1,19 @@
+// Copyright (c) The Thanos Authors.
+// Licensed under the Apache License 2.0.
+
+package logging
+
+import (
+	"testing"
+
+	"github.com/go-kit/log/level"
+)
+
+func BenchmarkDisallowedLogLevels(b *testing.B) {
+	logger := NewLogger("warn", "logfmt", "benchmark")
+
+	for i := 0; i < b.N; i++ {
+		level.Info(logger).Log("hello", "world", "number", i)
+		level.Debug(logger).Log("hello", "world", "number", i)
+	}
+}