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

MP metrics does not correctly rescale distributions (e.g., timers, histograms) #3731

Closed
2 tasks done
tjquinno opened this issue Dec 13, 2021 · 0 comments · Fixed by #3733
Closed
2 tasks done

MP metrics does not correctly rescale distributions (e.g., timers, histograms) #3731

tjquinno opened this issue Dec 13, 2021 · 0 comments · Fixed by #3733
Assignees
Labels
2.x Issues for 2.x version branch 3.x Issues for 3.x version branch bug Something isn't working metrics MP P2
Projects

Comments

@tjquinno
Copy link
Member

tjquinno commented Dec 13, 2021

Environment Details

  • Helidon Version: 2.x
  • Helidon SE or Helidon MP MP
  • JDK version:
  • OS:
  • Docker version (if applicable):

Problem Description

In MP applications, the periodic calculation of the current time in seconds does not occur properly.

Steps to reproduce

(easiest way)

  1. Generate the Helidon MP QuickStart app. Add a @Timed annotation to the GreetResource#getMessage method.
  2. Build the project with mvn package
  3. Run with the debugger: java -agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=4000 -jar target/helidon-quickstart-mp.jar
  4. Using an IDE, set a breakpoint at ExponentiallyDecayingReservoir#updateTimeInSeconds and connect to the paused server to allow it to continue.
  5. You will hit the breakpoint once as the system initializes but not after that. The breakpoint should fire a few times each second.
@tjquinno tjquinno added bug Something isn't working MP metrics 2.x Issues for 2.x version branch 3.x Issues for 3.x version branch labels Dec 13, 2021
@tjquinno tjquinno self-assigned this Dec 13, 2021
@tjquinno tjquinno added this to Triage in Backlog via automation Dec 13, 2021
@tjquinno tjquinno added the P2 label Dec 13, 2021
@tjquinno tjquinno moved this from Triage to In Progress in Backlog Dec 13, 2021
Backlog automation moved this from In Progress to Closed Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Issues for 2.x version branch 3.x Issues for 3.x version branch bug Something isn't working metrics MP P2
Projects
Backlog
  
Closed
Development

Successfully merging a pull request may close this issue.

1 participant