From 9f15a0a9ac54e412013c567144ade5bc63d8d7dd Mon Sep 17 00:00:00 2001 From: Denis Khalikov Date: Sun, 9 Jul 2023 17:31:01 +0300 Subject: [PATCH] MCOL-5522 Temporary disable join match limit on BPP Current join pipeline is not designed to send a temporal result and continue execution. RowGroup for large side is the same where we store the matched rows, we cannot continue to iterate over it, requires a proper refactoring. --- primitives/primproc/batchprimitiveprocessor.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/primitives/primproc/batchprimitiveprocessor.cpp b/primitives/primproc/batchprimitiveprocessor.cpp index 11849a647..e2a03757b 100644 --- a/primitives/primproc/batchprimitiveprocessor.cpp +++ b/primitives/primproc/batchprimitiveprocessor.cpp @@ -1409,8 +1409,9 @@ uint32_t BatchPrimitiveProcessor::executeTupleJoin(uint32_t startRid) // The caller will restart to continue where we left off. if (resultCount >= maxResultCount) { - newStartRid += newRowCount; - break; + // FIXME: Implement proper pipleline. (MCOL-5522). + cerr << "BPP join match count exceeded the limit, match count: " << resultCount << endl; + resultCount = 0; } }