Skip to content

Commit

Permalink
changefeedccl: updated retryable error warning message
Browse files Browse the repository at this point in the history
Retryable error warning message contained the word "error"
Confusing to users because warning message had the word "error" in it
Prefaced warning message with "WARNING"

Release note (enterprise change): updated retyable error warning message to begin with "WARNING"
  • Loading branch information
Alex Wong committed Sep 14, 2021
1 parent 386e099 commit 68bb9a9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/ccl/changefeedccl/changefeed_stmt.go
Original file line number Diff line number Diff line change
Expand Up @@ -734,7 +734,7 @@ func (b *changefeedResumer) resumeWithRetries(
return err
}

log.Warningf(ctx, `CHANGEFEED job %d encountered retryable error: %v`, jobID, err)
log.Warningf(ctx, `WARNING: CHANGEFEED job %d encountered retryable error: %v`, jobID, err)
b.setJobRunningStatus(ctx, "retryable error: %s", err)
if metrics, ok := execCfg.JobRegistry.MetricsStruct().Changefeed.(*Metrics); ok {
metrics.ErrorRetries.Inc(1)
Expand Down

0 comments on commit 68bb9a9

Please sign in to comment.