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

Removed some hot path logs #7846

Merged
merged 2 commits into from Feb 14, 2020
Merged

Removed some hot path logs #7846

merged 2 commits into from Feb 14, 2020

Conversation

@agr
Copy link
Contributor

agr commented Feb 14, 2020

This was logging every time a search service request was sent. This line alone produces 500k-1m log lines per hour and duplicates the dependencies logging that is populated automatically.

Fixed the ResilientSearchHttpClient constructor exceptions.
@agr agr changed the base branch from master to dev Feb 14, 2020
Copy link
Contributor

skofman1 left a comment

:shipit:

@agr agr merged commit f75e202 into dev Feb 14, 2020
2 checks passed
2 checks passed
NuGetGallery - CI #68139 succeeded
Details
license/cla All CLA requirements met.
Details
@agr agr deleted the agr-less-logs branch Feb 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.