Skip to content

Conversation

@a-nogikh
Copy link
Collaborator

After #6066, we have started to see a number of log messages about bench/corpus uploads aborted due to context cancelation. These messages are not of any value.

Ignore Errorf() calls where one of the arguments encloses a context.Canceled error. This is probably a bit hacky, but at least prevents repeating errors.Is(err, context.Canceled) for every call that may be canceled.

After google#6066, we have started to see a number of log messages about
bench/corpus uploads aborted due to context cancelation. These messages
are not of any value.

Ignore Errorf() calls where one of the arguments encloses a
context.Canceled error. This is probably a bit hacky, but at least
prevents repeating errors.Is(err, context.Canceled) for every call that
may be canceled.
@a-nogikh a-nogikh requested a review from melver June 10, 2025 09:58
@a-nogikh a-nogikh added this pull request to the merge queue Jun 10, 2025
Merged via the queue into google:master with commit 8937fbd Jun 10, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants