Skip to content

Commit 3f588e5

Browse files
authored
refactor: Deprecate the old cleanup aggregation node ctor APIs (#26973)
``` == NO RELEASE NOTE == ``` Differential Revision: D90807059
1 parent db41f88 commit 3f588e5

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

presto-native-execution/presto_cpp/main/types/PrestoToVeloxQueryPlan.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -870,6 +870,7 @@ VeloxQueryPlanConverterBase::toColumnStatsSpec(
870870
aggregateNames,
871871
aggregates,
872872
/*ignoreNullKeys=*/false,
873+
/*noGroupsSpanBatches=*/false,
873874
sourceVeloxPlan);
874875

875876
// Sanity checks on aggregation node.
@@ -1104,6 +1105,7 @@ VeloxQueryPlanConverterBase::toVeloxQueryPlan(
11041105
globalGroupingSets,
11051106
groupIdField,
11061107
/*ignoreNullKeys=*/false,
1108+
/*noGroupsSpanBatches=*/false,
11071109
toVeloxQueryPlan(node->source, tableWriteInfo, taskId));
11081110
}
11091111

@@ -1184,6 +1186,7 @@ VeloxQueryPlanConverterBase::toVeloxQueryPlan(
11841186
/*aggregateNames=*/std::vector<std::string>{},
11851187
/*aggregates=*/std::vector<core::AggregationNode::Aggregate>{},
11861188
/*ignoreNullKeys=*/false,
1189+
/*noGroupsSpanBatches=*/false,
11871190
toVeloxQueryPlan(node->source, tableWriteInfo, taskId)));
11881191
}
11891192

0 commit comments

Comments
 (0)