-
Notifications
You must be signed in to change notification settings - Fork 113
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
Deduplicate provider logs #558
Conversation
585c921
to
2aecd32
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a couple of comments to think on. Thanks for removing the batching, that made this PR easier to review - given that this PR is only dedup'ing logs now, we should update the title as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some misc. thoughts:
- Instead of changing all the call sites of
createAwaitConfig#logStatus
toconfig.logger.LogMessage
, it seems like we could just changelogStatus
to log to theLogger
itself. Seems like it would cut the diff down significantly?
f97986d
to
b755a3c
Compare
Fixes #518
awaiters to update