Fix TraceQL results caching bug for floats ending in .0

Author: carles-grafana

CHANGELOG.md

@@ -1,5 +1,7 @@
 ## main / unreleased
 
+* [BUGFIX] TraceQL results caching bug for floats ending in .0 [#4539](https://github.com/grafana/tempo/pull/4539) (@carles-grafana)
+
 # v2.7.0-rc.0
 
 * [CHANGE] Disable gRPC compression in the querier and distributor for performance reasons [#4429](https://github.com/grafana/tempo/pull/4429) (@carles-grafana)

pkg/traceql/ast_stringer.go

@@ -89,7 +89,12 @@ func (s Static) EncodeToString(quotes bool) string {
 		i, _ := s.Int()
 		return strconv.Itoa(i)
 	case TypeFloat:
-		return strconv.FormatFloat(s.Float(), 'g', -1, 64)
+		f := strconv.FormatFloat(s.Float(), 'g', -1, 64)
+		// if the float string doesn't contain e or ., then append .0 to distinguish it from an int
+		if !strings.ContainsAny(f, "e.") {
+			f = f + ".0"
+		}
+		return f
 	case TypeString:
 		var str string
 		if len(s.valBytes) > 0 {

pkg/traceql/ast_test.go

@@ -131,9 +131,9 @@ func TestStatic_String(t *testing.T) {
 		{arg: nil, want: "nil"},
 		{arg: 101, want: "101"},
 		{arg: -10, want: "-10"},
-		{arg: -1.0, want: "-1"},
-		{arg: 0.0, want: "0"},
-		{arg: 10.0, want: "10"},
+		{arg: -1.0, want: "-1.0"},
+		{arg: 0.0, want: "0.0"},
+		{arg: 10.0, want: "10.0"},
 		{arg: "test", want: "`test`"},
 		{arg: true, want: "true"},
 		{arg: StatusOk, want: "ok"},

pkg/traceql/engine_metrics_test.go

@@ -352,7 +352,7 @@ func TestQuantileOverTime(t *testing.T) {
 	// Output series with quantiles per foo
 	// Prom labels are sorted alphabetically, traceql labels maintain original order.
 	out := SeriesSet{
-		`{p="0", span.foo="bar"}`: TimeSeries{
+		`{p="0.0", span.foo="bar"}`: TimeSeries{
 			Labels: []Label{
 				{Name: "span.foo", Value: NewStaticString("bar")},
 				{Name: "p", Value: NewStaticFloat(0)},
@@ -374,14 +374,14 @@ func TestQuantileOverTime(t *testing.T) {
 				0,
 			},
 		},
-		`{p="1", span.foo="bar"}`: TimeSeries{
+		`{p="1.0", span.foo="bar"}`: TimeSeries{
 			Labels: []Label{
 				{Name: "span.foo", Value: NewStaticString("bar")},
 				{Name: "p", Value: NewStaticFloat(1)},
 			},
 			Values: []float64{_512ns, _256ns, 0},
 		},
-		`{p="0", span.foo="baz"}`: TimeSeries{
+		`{p="0.0", span.foo="baz"}`: TimeSeries{
 			Labels: []Label{
 				{Name: "span.foo", Value: NewStaticString("baz")},
 				{Name: "p", Value: NewStaticFloat(0)},
@@ -401,7 +401,7 @@ func TestQuantileOverTime(t *testing.T) {
 				percentileHelper(0.5, _512ns, _512ns, _512ns),
 			},
 		},
-		`{p="1", span.foo="baz"}`: TimeSeries{
+		`{p="1.0", span.foo="baz"}`: TimeSeries{
 			Labels: []Label{
 				{Name: "span.foo", Value: NewStaticString("baz")},
 				{Name: "p", Value: NewStaticFloat(1)},

pkg/traceql/test_examples.yaml

@@ -54,6 +54,7 @@ valid:
   - '{ event.foo = "bar" }'
   - '{ link.foo = "bar" }'
   - '{ instrumentation.foo = "bar" }'
+  - '{ span.foo != 3.0 }'
   # scoped intrinsics
   - '{ trace:duration > 2s }'
   - '{ trace:rootName = "a" }'
@@ -151,7 +152,6 @@ valid:
   # undocumented - nested set
   - '{ nestedSetLeft > 3 }'
   - '{ } >> { kind = server } | select(nestedSetLeft, nestedSetRight, nestedSetParent)'
-  
 # parse_fails throw an error when parsing
 parse_fails:
   - 'true'