-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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 latency recording utility handler #9172
Add latency recording utility handler #9172
Conversation
Got an example of how it would be used? |
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.
Just some niggles that you can fix if you like, otherwise good.
...y-server/src/main/java/org/eclipse/jetty/server/handler/AbstractLatencyRecordingHandler.java
Outdated
Show resolved
Hide resolved
...y-server/src/main/java/org/eclipse/jetty/server/handler/AbstractLatencyRecordingHandler.java
Outdated
Show resolved
Hide resolved
@joakime The fact that you made this comment means a test is needed, at least as some internal doc showing how this works. I've now added such test. |
...y-server/src/main/java/org/eclipse/jetty/server/handler/AbstractLatencyRecordingHandler.java
Outdated
Show resolved
Hide resolved
1cd5831
to
451ead8
Compare
Signed-off-by: Ludovic Orban <[email protected]>
Signed-off-by: Ludovic Orban <[email protected]>
451ead8
to
4977f7d
Compare
Signed-off-by: Ludovic Orban <[email protected]>
4977f7d
to
be93d13
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.
that's better!
Add a small utility handler that standardizes the way latencies should be recorded.
Closes #8069.