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

Add support for async servlet to metrics-servlet #796

Merged
merged 1 commit into from Jun 12, 2015

Conversation

Projects
None yet
3 participants
@mlaflamm

mlaflamm commented May 7, 2015

Proposition for issue #795.

The requests timer and the activeRequests counter now properly observe the entire processing time. Use meters separate from the response code meters for timeouts and errors since the code isn't representative (i.e. 200) for these situations.

Manuel Laflamme
Add support for async servlet to metrics-servlet
The requests timer and the activeRequests counter now properly observe the entire processing time. Use meters separate from the response code meters for timeouts and errors since the code isn't representative (i.e. 200) for these situations.

ryantenney added a commit that referenced this pull request Jun 12, 2015

Merge pull request #796 from mlaflamm/metrics-servlet_795
Add support for async servlet to metrics-servlet

@ryantenney ryantenney merged commit 3542e71 into dropwizard:master Jun 12, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ryantenney

This comment has been minimized.

Show comment
Hide comment
@ryantenney

ryantenney Jun 12, 2015

Member

Merged, thanks!

Member

ryantenney commented Jun 12, 2015

Merged, thanks!

@ryantenney ryantenney added this to the 4.0.0 milestone Jun 12, 2015

@arteam arteam modified the milestones: 4.0.0, 3.2.0 Dec 24, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment