-marks absent as non parallelizable

owen-d · owen-d · commit 6abe6a24d011 · 2020-02-05T08:29:03.000-05:00
Signed-off-by: Owen Diehl &lt;ow.diehl@gmail.com&gt;
diff --git a/pkg/querier/astmapper/parallel.go b/pkg/querier/astmapper/parallel.go
@@ -17,6 +17,7 @@ var summableAggregates = map[promql.ItemType]struct{}{
 var nonParallelFuncs = []string{
 	"histogram_quantile",
 	"quantile_over_time",
+	"absent",
 }
 
 // CanParallel tests if a subtree is parallelizable.
diff --git a/pkg/querier/queryrange/promql_test.go b/pkg/querier/queryrange/promql_test.go
@@ -297,9 +297,6 @@ func Test_FunctionParallelism(t *testing.T) {
 		{
 			fn: "abs",
 		},
-		{
-			fn: "absent",
-		},
 		{
 			fn:           "avg_over_time",
 			isTestMatrix: true,

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ var summableAggregates = map[promql.ItemType]struct{}{`
`17`	`17`	`var nonParallelFuncs = []string{`
`18`	`18`	`"histogram_quantile",`
`19`	`19`	`"quantile_over_time",`
	`20`	`+ "absent",`
`20`	`21`	`}`
`21`	`22`
`22`	`23`	`// CanParallel tests if a subtree is parallelizable.`
Original file line number	Diff line number	Diff line change
`@@ -297,9 +297,6 @@ func Test_FunctionParallelism(t *testing.T) {`
`297`	`297`	`{`
`298`	`298`	`fn: "abs",`
`299`	`299`	`},`
`300`		`- {`
`301`		`- fn: "absent",`
`302`		`- },`
`303`	`300`	`{`
`304`	`301`	`fn: "avg_over_time",`
`305`	`302`	`isTestMatrix: true,`