Skip to content

Commit 00bbd5a

Browse files
authored
executor: Remove useless error logs for indexLoopUp executor when context canceled (#61073) (#61114)
close #61072
1 parent 003692e commit 00bbd5a

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

pkg/executor/distsql.go

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -890,7 +890,9 @@ func (e *IndexLookUpExecutor) buildTableReader(ctx context.Context, task *lookup
890890
tableReaderExec.buildVirtualColumnInfo()
891891
tableReader, err := e.dataReaderBuilder.buildTableReaderFromHandles(ctx, tableReaderExec, task.handles, true)
892892
if err != nil {
893-
logutil.Logger(ctx).Error("build table reader from handles failed", zap.Error(err))
893+
if ctx.Err() != context.Canceled {
894+
logutil.Logger(ctx).Error("build table reader from handles failed", zap.Error(err))
895+
}
894896
return nil, err
895897
}
896898
return tableReader, nil
@@ -1533,7 +1535,9 @@ func (w *tableWorker) executeTask(ctx context.Context, task *lookupTableTask) er
15331535
tableReader, err := w.idxLookup.buildTableReader(ctx, task)
15341536
task.buildDoneTime = time.Now()
15351537
if err != nil {
1536-
logutil.Logger(ctx).Error("build table reader failed", zap.Error(err))
1538+
if ctx.Err() != context.Canceled {
1539+
logutil.Logger(ctx).Error("build table reader failed", zap.Error(err))
1540+
}
15371541
return err
15381542
}
15391543
defer func() { terror.Log(exec.Close(tableReader)) }()
@@ -1564,7 +1568,9 @@ func (w *tableWorker) executeTask(ctx context.Context, task *lookupTableTask) er
15641568
chk := exec.TryNewCacheChunk(tableReader)
15651569
err = exec.Next(ctx, tableReader, chk)
15661570
if err != nil {
1567-
logutil.Logger(ctx).Error("table reader fetch next chunk failed", zap.Error(err))
1571+
if ctx.Err() != context.Canceled {
1572+
logutil.Logger(ctx).Error("table reader fetch next chunk failed", zap.Error(err))
1573+
}
15681574
return err
15691575
}
15701576
if chk.NumRows() == 0 {

0 commit comments

Comments
 (0)