summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorlucius <[email protected]>2025-07-02 12:33:22 +0300
committerlucius <[email protected]>2025-07-02 12:48:21 +0300
commit7254af9393639e2abb0986cc9d84c4cfd37ae1f1 (patch)
treed3b73d7eb5c4cabf3d788c496b0bd28a43e7c823
parent76d96655b1b66ba56477f897e1a7d00ffa217a6b (diff)
YQL-20085 fix inputs order in ExpandEquiJoin part2
commit_hash:257767d7294904de2f60fea5587871d661ea0efa
-rw-r--r--yql/essentials/core/peephole_opt/yql_opt_peephole_physical.cpp5
-rw-r--r--yql/essentials/tests/s-expressions/minirun/part8/canondata/result.json6
2 files changed, 3 insertions, 8 deletions
diff --git a/yql/essentials/core/peephole_opt/yql_opt_peephole_physical.cpp b/yql/essentials/core/peephole_opt/yql_opt_peephole_physical.cpp
index 2b7da0e7530..bdbceeb0a33 100644
--- a/yql/essentials/core/peephole_opt/yql_opt_peephole_physical.cpp
+++ b/yql/essentials/core/peephole_opt/yql_opt_peephole_physical.cpp
@@ -723,11 +723,6 @@ TExprNode::TPtr ExpandEquiJoinImpl(const TExprNode& node, TExprContext& ctx) {
std::vector<std::string_view> lKeys(keyMembers1.size()), rKeys(keyMembers2.size());
MKQL_ENSURE(keyMembers1.size() == keyMembers2.size(), "Expected same key sizes.");
- for (ui32 i = 0; i < keyMembers1.size(); ++i) {
- if (keyMembers1Inputs[i] != 0) {
- std::swap(keyMembers1[i], keyMembers2[i]);
- }
- }
bool optKey = false, badKey = false;
const bool filter = joinKind == "Inner" || joinKind.ends_with("Semi");
diff --git a/yql/essentials/tests/s-expressions/minirun/part8/canondata/result.json b/yql/essentials/tests/s-expressions/minirun/part8/canondata/result.json
index bf0df4335ce..6fe3fbc2e7f 100644
--- a/yql/essentials/tests/s-expressions/minirun/part8/canondata/result.json
+++ b/yql/essentials/tests/s-expressions/minirun/part8/canondata/result.json
@@ -176,9 +176,9 @@
],
"test.test[EquiJoin-YQL-20085-default.txt-Results]": [
{
- "checksum": "8b65cde7720bf12d6ef07af1add1e683",
- "size": 31040,
- "uri": "https://{canondata_backend}/1937027/d4a75065adb37091f0071155d118edcd18ce277c/resource.tar.gz#test.test_EquiJoin-YQL-20085-default.txt-Results_/results.txt"
+ "checksum": "91d704be27ac65565be129ad9d4beeac",
+ "size": 33372,
+ "uri": "https://{canondata_backend}/1937150/442e463e9e2a721d540e63b473422a22a4d9dbca/resource.tar.gz#test.test_EquiJoin-YQL-20085-default.txt-Results_/results.txt"
}
],
"test.test[File-DifferentAliasesToSameFiles--Debug]": [