executor: fix an issue that index_lookup_hash_join incorrectly estimate memory usage. (#47795) (#47920)

ti-chi-bot · web-flow · commit 1ebb9b2fe48b · 2023-10-23T10:59:05.000Z
close #47788
diff --git a/executor/builder.go b/executor/builder.go
@@ -4137,7 +4137,7 @@ func buildKvRangesForIndexJoin(ctx sessionctx.Context, tableID, indexID int64, l
 		memTracker.Consume(int64(2 * cap(kvRanges[0].StartKey) * len(kvRanges)))
 	}
 	if len(tmpDatumRanges) != 0 && memTracker != nil {
-		memTracker.Consume(2 * int64(len(tmpDatumRanges)) * types.EstimatedMemUsage(tmpDatumRanges[0].LowVal, len(tmpDatumRanges)))
+		memTracker.Consume(2 * types.EstimatedMemUsage(tmpDatumRanges[0].LowVal, len(tmpDatumRanges)))
 	}
 	if cwc == nil {
 		sort.Slice(kvRanges, func(i, j int) bool {

Original file line number	Diff line number	Diff line change
`@@ -4137,7 +4137,7 @@ func buildKvRangesForIndexJoin(ctx sessionctx.Context, tableID, indexID int64, l`
`4137`	`4137`	`memTracker.Consume(int64(2 * cap(kvRanges[0].StartKey) * len(kvRanges)))`
`4138`	`4138`	`}`
`4139`	`4139`	`if len(tmpDatumRanges) != 0 && memTracker != nil {`
`4140`		`- memTracker.Consume(2 * int64(len(tmpDatumRanges)) * types.EstimatedMemUsage(tmpDatumRanges[0].LowVal, len(tmpDatumRanges)))`
	`4140`	`+ memTracker.Consume(2 * types.EstimatedMemUsage(tmpDatumRanges[0].LowVal, len(tmpDatumRanges)))`
`4141`	`4141`	`}`
`4142`	`4142`	`if cwc == nil {`
`4143`	`4143`	`sort.Slice(kvRanges, func(i, j int) bool {`