-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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 guide for MicroProfile Metrics #1548
Conversation
3caf91d
to
54fb5ab
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.
@jmartisk I added a bunch of comments. Could you apply them (or give me some feedback if you disagree) and squash?
Thanks!
} | ||
``` | ||
|
||
Description what each metric means: |
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.
These metrics are worth an explanation:
would be better I think
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.
All of them are explained there, not just some. In some cases, multiple values are just squashed into one line to make it shorter.
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.
Yeah, agreed. I'm just reformulating what you're saying. You can use Those
or All those
if you prefer :).
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.
Or Let's explain what each metric means:
54fb5ab
to
736350f
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.
I went with a variation of my latest proposal which is very close to your initial proposal.
Will merge, thanks!
Guide for the quickstart being added through quarkusio/quarkus-quickstarts#101