Skip to content

Commit

Permalink
Merge #62584
Browse files Browse the repository at this point in the history
62584: sql: remove unused error r=yuzefovich a=yuzefovich

Release note: None

Co-authored-by: Yahor Yuzefovich <[email protected]>
  • Loading branch information
craig[bot] and yuzefovich committed Mar 26, 2021
2 parents 56ef155 + 1d2fadf commit 1c9289e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
1 change: 0 additions & 1 deletion pkg/sql/distsql_running.go
Original file line number Diff line number Diff line change
Expand Up @@ -371,7 +371,6 @@ func (dsp *DistSQLPlanner) Run(
// We need to close the planNode tree we translated into a DistSQL plan
// before flow.Cleanup, which closes memory accounts that expect to be
// emptied.
curPlan.execErr = recv.resultWriter.Err()
curPlan.close(ctx)
flow.Cleanup(ctx)
}
Expand Down
3 changes: 0 additions & 3 deletions pkg/sql/plan.go
Original file line number Diff line number Diff line change
Expand Up @@ -300,9 +300,6 @@ type planTop struct {
// flags is populated during planning and execution.
flags planFlags

// execErr retains the last execution error, if any.
execErr error

// avoidBuffering, when set, causes the execution to avoid buffering
// results.
avoidBuffering bool
Expand Down

0 comments on commit 1c9289e

Please sign in to comment.