Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

stopper: downgrade a spammy log message to a log event #24516

Merged

Conversation

petermattis
Copy link
Collaborator

Downgrade a log message about "throttling async task" to a log
event. This occurs easily for the timeseries usage of RunLimitedAsync
task.

Fixes #24512

Release note: None

Downgrade a log message about "throttling async task" to a log
event. This occurs easily for the timeseries usage of `RunLimitedAsync`
task.

Fixes cockroachdb#24512

Release note: None
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@petermattis petermattis requested a review from bdarnell April 5, 2018 17:46
@petermattis
Copy link
Collaborator Author

bors r=bdarnell

craig bot added a commit that referenced this pull request Apr 5, 2018
24516: stopper: downgrade a spammy log message to a log event r=bdarnell a=petermattis

Downgrade a log message about "throttling async task" to a log
event. This occurs easily for the timeseries usage of `RunLimitedAsync`
task.

Fixes #24512

Release note: None
@craig
Copy link
Contributor

craig bot commented Apr 5, 2018

Build succeeded

@craig craig bot merged commit 073bb37 into cockroachdb:master Apr 5, 2018
@petermattis petermattis deleted the pmattis/silence-spammy-log-message branch April 5, 2018 18:50
craig bot added a commit that referenced this pull request Apr 5, 2018
24522: backport-2.0: stopper: downgrade a spammy log message to a log event r=bdarnell a=petermattis

Backport 1/1 commits from #24516.

/cc @cockroachdb/release

---

Downgrade a log message about "throttling async task" to a log
event. This occurs easily for the timeseries usage of `RunLimitedAsync`
task.

Fixes #24512

Release note: None
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.

3 participants