Skip to content

Commit 7e8bc8a

Browse files
committed
sql: lazily allocate includeIndexes
Release note: None
1 parent a33d5c5 commit 7e8bc8a

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

pkg/sql/row/updater.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ func MakeUpdater(
116116
}
117117

118118
updateColIDToRowIndex := ColIDtoRowIndexFromCols(updateCols)
119-
includeIndexes := make([]catalog.Index, 0, len(tableDesc.WritableNonPrimaryIndexes()))
119+
var includeIndexes []catalog.Index
120120
var deleteOnlyIndexes []catalog.Index
121121
// If the UPDATE is set to only update columns, do not collect secondary
122122
// indexes to update.
@@ -128,6 +128,9 @@ func MakeUpdater(
128128
continue
129129
}
130130
if !index.DeleteOnly() {
131+
if includeIndexes == nil {
132+
includeIndexes = make([]catalog.Index, 0, len(tableDesc.WritableNonPrimaryIndexes()))
133+
}
131134
includeIndexes = append(includeIndexes, index)
132135
} else {
133136
if deleteOnlyIndexes == nil {

0 commit comments

Comments
 (0)