From 9dd868c5bd625531fa594dc1668ab37f0bd07a93 Mon Sep 17 00:00:00 2001 From: Jordan Lewis Date: Thu, 30 Aug 2018 17:21:09 -0400 Subject: [PATCH] sql: don't remove batch limit in local lookup join Previously, the local lookup join node was created with an underlying scan with an explicitly disabled batch limit. This is the wrong thing to do, since that scan will be used as a full table scan - it needs a batch limit to prevent OOM errors. Release note: None --- pkg/sql/opt_exec_factory.go | 1 - 1 file changed, 1 deletion(-) diff --git a/pkg/sql/opt_exec_factory.go b/pkg/sql/opt_exec_factory.go index 1810487c2689..26ed3e7fc9a8 100644 --- a/pkg/sql/opt_exec_factory.go +++ b/pkg/sql/opt_exec_factory.go @@ -553,7 +553,6 @@ func (ef *execFactory) ConstructLookupJoin( tableScan.index = indexDesc tableScan.run.isSecondaryIndex = (indexDesc != &tabDesc.PrimaryIndex) - tableScan.disableBatchLimit() n := &lookupJoinNode{ input: input.(planNode),