Skip to content

Commit b44cd79

Browse files
canonize plans
1 parent fb56eb6 commit b44cd79

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

ydb/core/kqp/ut/olap/aggregations_ut.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ Y_UNIT_TEST_SUITE(KqpOlapAggregations) {
163163

164164
// Check plan
165165
#if SSA_RUNTIME_VERSION >= 2U
166-
CheckPlanForAggregatePushdown(query, tableClient, { "WideCombiner" }, "Aggregate-TableFullScan");
166+
CheckPlanForAggregatePushdown(query, tableClient, { "WideCombiner" }, "TableFullScan");
167167
// CheckPlanForAggregatePushdown(query, tableClient, { "TKqpOlapAgg" }, "TableFullScan");
168168
#else
169169
CheckPlanForAggregatePushdown(query, tableClient, { "CombineCore" }, "");
@@ -980,7 +980,7 @@ Y_UNIT_TEST_SUITE(KqpOlapAggregations) {
980980
ORDER BY c, resource_id DESC LIMIT 3
981981
)")
982982
.SetExpectedReply("[[[\"40999\"];[4];1u];[[\"40998\"];[3];1u];[[\"40997\"];[2];1u]]")
983-
.SetExpectedReadNodeType("Aggregate-TableFullScan");
983+
.SetExpectedReadNodeType("TableFullScan");
984984
testCase.FillExpectedAggregationGroupByPlanOptions();
985985
TestAggregations({ testCase });
986986
}

ydb/core/kqp/ut/olap/clickbench_ut.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ Y_UNIT_TEST_SUITE(KqpOlapClickbench) {
155155
//.SetExpectedReply("[[[\"40999\"];[4];1u];[[\"40998\"];[3];1u];[[\"40997\"];[2];1u]]")
156156
// Should be fixed in https://st.yandex-team.ru/KIKIMR-17009
157157
// .SetExpectedReadNodeType("TableFullScan");
158-
.SetExpectedReadNodeType("Aggregate-TableFullScan");
158+
.SetExpectedReadNodeType("TableFullScan");
159159
q7.FillExpectedAggregationGroupByPlanOptions();
160160

161161
TAggregationTestCase q9;
@@ -185,7 +185,7 @@ Y_UNIT_TEST_SUITE(KqpOlapClickbench) {
185185
//.SetExpectedReply("[[[\"40999\"];[4];1u];[[\"40998\"];[3];1u];[[\"40997\"];[2];1u]]")
186186
// Should be fixed in https://st.yandex-team.ru/KIKIMR-17009
187187
// .SetExpectedReadNodeType("TableFullScan");
188-
.SetExpectedReadNodeType("Aggregate-TableFullScan");
188+
.SetExpectedReadNodeType("TableFullScan");
189189
q12.FillExpectedAggregationGroupByPlanOptions();
190190

191191
TAggregationTestCase q14;
@@ -201,7 +201,7 @@ Y_UNIT_TEST_SUITE(KqpOlapClickbench) {
201201
//.SetExpectedReply("[[[\"40999\"];[4];1u];[[\"40998\"];[3];1u];[[\"40997\"];[2];1u]]")
202202
// Should be fixed in https://st.yandex-team.ru/KIKIMR-17009
203203
// .SetExpectedReadNodeType("TableFullScan");
204-
.SetExpectedReadNodeType("Aggregate-TableFullScan");
204+
.SetExpectedReadNodeType("TableFullScan");
205205
q14.FillExpectedAggregationGroupByPlanOptions();
206206

207207
TAggregationTestCase q22;

ydb/core/kqp/ut/olap/kqp_olap_ut.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2520,7 +2520,8 @@ Y_UNIT_TEST_SUITE(KqpOlap) {
25202520
UNIT_ASSERT_EQUAL_C(plan.QueryStats->Getquery_ast().find("WideFromBlocks"), plan.QueryStats->Getquery_ast().rfind("WideFromBlocks"), plan.QueryStats->Getquery_ast());
25212521
break;
25222522
case NKikimrConfig::TTableServiceConfig_EBlockChannelsMode_BLOCK_CHANNELS_FORCE:
2523-
UNIT_ASSERT_C(plan.QueryStats->Getquery_ast().Contains("return (FromFlow (WideMap"), plan.QueryStats->Getquery_ast());
2523+
UNIT_ASSERT_C(plan.QueryStats->Getquery_ast().Contains("(FromFlow (WideSortBlocks"), plan.QueryStats->Getquery_ast());
2524+
UNIT_ASSERT_C(plan.QueryStats->Getquery_ast().Contains("(FromFlow (NarrowMap (WideFromBlocks"), plan.QueryStats->Getquery_ast());
25242525
break;
25252526
}
25262527
}

0 commit comments

Comments
 (0)