Revert "Fixed a problem with plans in PG ut (#1974)" (#2020)

pavelvelikhov · web-flow · commit e41835d0d997 · 2024-02-16T17:42:48.000+03:00
diff --git a/ydb/core/kqp/opt/kqp_query_plan.cpp b/ydb/core/kqp/opt/kqp_query_plan.cpp
@@ -1881,13 +1881,9 @@ void BuildPlanIndex(NJson::TJsonValue& plan, THashMap<int, NJson::TJsonValue>& p
 
         auto pos = precomputeName.find("precompute");
         if (pos != TString::npos) {
-            if (!precomputes.contains(precomputeName.substr(pos))) {
-                precomputes[precomputeName.substr(pos)] = plan;
-            }
+            precomputes[precomputeName.substr(pos)] = plan;
         } else if (precomputeName.size()>=4 && precomputeName.find("CTE ") != TString::npos) {
-            if (!precomputes.contains(precomputeName.substr(4))) {
-                precomputes[precomputeName.substr(4)] = plan;
-            }
+            precomputes[precomputeName.substr(4)] = plan;
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1881,13 +1881,9 @@ void BuildPlanIndex(NJson::TJsonValue& plan, THashMap<int, NJson::TJsonValue>& p`
`1881`	`1881`
`1882`	`1882`	`auto pos = precomputeName.find("precompute");`
`1883`	`1883`	`if (pos != TString::npos) {`
`1884`		`- if (!precomputes.contains(precomputeName.substr(pos))) {`
`1885`		`- precomputes[precomputeName.substr(pos)] = plan;`
`1886`		`- }`
	`1884`	`+ precomputes[precomputeName.substr(pos)] = plan;`
`1887`	`1885`	`} else if (precomputeName.size()>=4 && precomputeName.find("CTE ") != TString::npos) {`
`1888`		`- if (!precomputes.contains(precomputeName.substr(4))) {`
`1889`		`- precomputes[precomputeName.substr(4)] = plan;`
`1890`		`- }`
	`1886`	`+ precomputes[precomputeName.substr(4)] = plan;`
`1891`	`1887`	`}`
`1892`	`1888`	`}`
`1893`	`1889`