Skip to content

Commit

Permalink
fix: Fix the driver concurrency setting in test
Browse files Browse the repository at this point in the history
Summary: The tests use a non-existing query config

Differential Revision: D66994115
  • Loading branch information
xiaoxmeng authored and facebook-github-bot committed Dec 10, 2024
1 parent b9cce6d commit 8f6ef57
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions velox/exec/tests/AggregationTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -941,7 +941,7 @@ TEST_F(AggregationTest, partialDistinctWithAbandon) {
auto task = AssertQueryBuilder(duckDbQueryRunner_)
.config(QueryConfig::kAbandonPartialAggregationMinRows, 100)
.config(QueryConfig::kAbandonPartialAggregationMinPct, 50)
.config("max_drivers_per_task", 1)
.maxDrivers(1)
.plan(PlanBuilder()
.values(vectors)
.partialAggregation({"c0"}, {})
Expand All @@ -953,7 +953,7 @@ TEST_F(AggregationTest, partialDistinctWithAbandon) {
task = AssertQueryBuilder(duckDbQueryRunner_)
.config(QueryConfig::kAbandonPartialAggregationMinRows, 100)
.config(QueryConfig::kAbandonPartialAggregationMinPct, 50)
.config("max_drivers_per_task", 1)
.maxDrivers(1)
.plan(PlanBuilder()
.values(vectors)
.partialAggregation({"c0"}, {"sum(c0)"})
Expand Down Expand Up @@ -990,7 +990,7 @@ TEST_F(AggregationTest, distinctWithGroupingKeysReordered) {
.config(QueryConfig::kSpillEnabled, true)
.config(QueryConfig::kAggregationSpillEnabled, true)
.config(QueryConfig::kSpillPrefixSortEnabled, true)
.config("max_drivers_per_task", 1)
.maxDrivers(1)
.plan(PlanBuilder()
.values(vectors)
.singleAggregation({"c2", "c0"}, {})
Expand Down

0 comments on commit 8f6ef57

Please sign in to comment.