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

Clean up redundant error messages. #847

Merged

Conversation

Projects
None yet
2 participants
@scrosby
Copy link
Member

commented Jul 10, 2019

We log both the thrown error as well as the log/error.

Changes proposed in this PR

Why are we making these changes?

Scott Crosby
Clean up redundant error messages.
We log both the thrown error as well as the log/error.

@scrosby scrosby force-pushed the scrosby:outgoing/clean-redundant-error-messages branch from 0cab4cd to ba89b56 Jul 10, 2019

Show resolved Hide resolved waiter/src/waiter/service.clj Outdated
Update waiter/src/waiter/service.clj
Co-Authored-By: Steven Radack <sradack@gmail.com>

@sradack sradack merged commit efd6a5c into twosigma:master Jul 11, 2019

1 of 2 checks passed

Mergeable Mergeable run returned Status ***FAIL***
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.