Skip to content

Commit f7149b7

Browse files
authored
executor: Remove useless error logs for indexLoopUp executor when context canceled (#61073)
close #61072
1 parent 2503d50 commit f7149b7

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
@@ -869,7 +869,9 @@ func (e *IndexLookUpExecutor) buildTableReader(ctx context.Context, task *lookup
869869
tableReaderExec.buildVirtualColumnInfo()
870870
tableReader, err := e.dataReaderBuilder.buildTableReaderFromHandles(ctx, tableReaderExec, task.handles, true)
871871
if err != nil {
872-
logutil.Logger(ctx).Error("build table reader from handles failed", zap.Error(err))
872+
if ctx.Err() != context.Canceled {
873+
logutil.Logger(ctx).Error("build table reader from handles failed", zap.Error(err))
874+
}
873875
return nil, err
874876
}
875877
return tableReader, nil
@@ -1531,7 +1533,9 @@ func (w *tableWorker) executeTask(ctx context.Context, task *lookupTableTask) er
15311533
tableReader, err := w.idxLookup.buildTableReader(ctx, task)
15321534
task.buildDoneTime = time.Now()
15331535
if err != nil {
1534-
logutil.Logger(ctx).Error("build table reader failed", zap.Error(err))
1536+
if ctx.Err() != context.Canceled {
1537+
logutil.Logger(ctx).Error("build table reader failed", zap.Error(err))
1538+
}
15351539
return err
15361540
}
15371541
defer func() { terror.Log(exec.Close(tableReader)) }()
@@ -1562,7 +1566,9 @@ func (w *tableWorker) executeTask(ctx context.Context, task *lookupTableTask) er
15621566
chk := exec.TryNewCacheChunk(tableReader)
15631567
err = exec.Next(ctx, tableReader, chk)
15641568
if err != nil {
1565-
logutil.Logger(ctx).Error("table reader fetch next chunk failed", zap.Error(err))
1569+
if ctx.Err() != context.Canceled {
1570+
logutil.Logger(ctx).Error("table reader fetch next chunk failed", zap.Error(err))
1571+
}
15661572
return err
15671573
}
15681574
if chk.NumRows() == 0 {

0 commit comments

Comments
 (0)