feat(detected-labels): include labels with cardinality > 1 (#13128)

svennergr · trevorwhitney · web-flow · commit 8be8364435bb · 2024-06-04T19:33:14.000Z
Co-authored-by: Trevor Whitney &lt;trevorjwhitney@gmail.com&gt;
diff --git a/pkg/querier/queryrange/roundtrip.go b/pkg/querier/queryrange/roundtrip.go
@@ -329,7 +329,7 @@ func NewDetectedLabelsCardinalityFilter(rt queryrangebase.Handler) queryrangebas
 			var result []*logproto.DetectedLabel
 
 			for _, dl := range resp.Response.DetectedLabels {
-				if dl.Cardinality > 2 && dl.Cardinality < 50 {
+				if dl.Cardinality > 1 && dl.Cardinality < 50 {
 					result = append(result, &logproto.DetectedLabel{Label: dl.Label, Cardinality: dl.Cardinality})
 				}
 			}

Original file line number	Diff line number	Diff line change
`@@ -329,7 +329,7 @@ func NewDetectedLabelsCardinalityFilter(rt queryrangebase.Handler) queryrangebas`
`329`	`329`	`var result []*logproto.DetectedLabel`
`330`	`330`
`331`	`331`	`for _, dl := range resp.Response.DetectedLabels {`
`332`		`- if dl.Cardinality > 2 && dl.Cardinality < 50 {`
	`332`	`+ if dl.Cardinality > 1 && dl.Cardinality < 50 {`
`333`	`333`	`result = append(result, &logproto.DetectedLabel{Label: dl.Label, Cardinality: dl.Cardinality})`
`334`	`334`	`}`
`335`	`335`	`}`