grace_join: redesign to improve LEFT JOIN with short left side (ydb-p…

…latform#8146)
Hor911 · Aug 28, 2024 · 419b953 · 419b953
1 parent 10af6ea
commit 419b953
Show file tree

Hide file tree

Showing 3 changed files with 123 additions and 433 deletions.
diff --git a/ydb/library/yql/minikql/comp_nodes/mkql_grace_join.cpp b/ydb/library/yql/minikql/comp_nodes/mkql_grace_join.cpp
@@ -589,7 +589,7 @@ class TGraceJoinSpillingSupportState : public TComputationValue<TGraceJoinSpilli
     ,   IsSpillingAllowed(isSpillingAllowed)
     {
         YQL_LOG(GRACEJOIN_DEBUG) << (const void *)&*JoinedTablePtr << "# AnyJoinSettings=" << (int)anyJoinSettings << " JoinKind=" << (int)joinKind;
-        if (JoinKind == EJoinKind::Full || JoinKind == EJoinKind::Exclusion || IsSelfJoin_) {
+        if (IsSelfJoin_) {
             LeftPacker->BatchSize = std::numeric_limits<ui64>::max();
             RightPacker->BatchSize = std::numeric_limits<ui64>::max();
         }