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

Fix LongTaskTimer output for LoggingMeterRegistry #5834

Closed
jonatan-ivanov opened this issue Jan 28, 2025 · 0 comments
Closed

Fix LongTaskTimer output for LoggingMeterRegistry #5834

jonatan-ivanov opened this issue Jan 28, 2025 · 0 comments
Labels
bug A general bug module: micrometer-core An issue that is related to our core module
Milestone

Comments

@jonatan-ivanov
Copy link
Member

jonatan-ivanov commented Jan 28, 2025

Creating and starting a simple LongTaskTimer:

registry.more().longTaskTimer("my.ltt").start();
// wait (1s)
// print

Results in an output like this:

my.ltt{} active=1 milliseconds duration=1s

where the active field should be the number of active tasks, so it should be unit less instead of milliseconds:

my.ltt{} active=1 duration=1s
@jonatan-ivanov jonatan-ivanov added bug A general bug module: micrometer-core An issue that is related to our core module labels Jan 28, 2025
@jonatan-ivanov jonatan-ivanov added this to the 1.13.11 milestone Jan 28, 2025
@jonatan-ivanov jonatan-ivanov changed the title Fix LongTaskTimer output for LoggingMeterRegistryTest Fix LongTaskTimer output for LoggingMeterRegistry Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A general bug module: micrometer-core An issue that is related to our core module
Projects
None yet
Development

No branches or pull requests

1 participant