-
Notifications
You must be signed in to change notification settings - Fork 82
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
Update New Relic docs #137
Conversation
Just wanted to leave a gentle poke to see if anybody might be able to look at this. Thanks. |
Bumping again. Would it be possible to get an estimate on when we might get eyes on this? 🙏 🙇♂️ |
Any updates? Critiques? Concerns? Feedback? 🍰 |
:toc: | ||
:sectnums: | ||
:system: new-relic | ||
|
||
New Relic offers a dimensional monitoring system product called Insights with a full UI and a query language called NRQL. New Relic Insights operates on a push model. Some features of NRQL assume that Insights receives a distinct event payload for every timing, count, etc. Micrometer instead ships aggregates at a prescribed interval, allowing your app's throughput to scale without concern for event propagation to Insights becoming a bottleneck. | ||
New Relic is a powerful observability platform with Micrometer |
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.
IMO, superlatives suck. All of the platforms micrometer supports are (differently) powerful. ;)
Squash down to one commit? @shakuzen and @jkschneider .. should the existing micrometer new relic extension be deprecated (and then have that reflected in the docs here?) |
Thank you for taking a look at this! Re: deprecating the existing registry -- there was some discussion a while back about New Relic possibly hosting it. That discussion is here: newrelic/micrometer-registry-newrelic#46. I would love for us to collectively come to an agreement on that point. The primary concern is that users might be confused about which one to use...so having a primary recommended solution is best. |
Thanks for the pull request. Sorry for the delay in responding here. I would like to have some response to newrelic/micrometer-registry-newrelic#46 (comment) before making these kinds of changes to our docs. I am in support of getting to a point of there being one New Relic registry implementation that works best for users. I am concerned about making such a transition as smooth as possible, and I think that's what needs to be worked out before we officially tell users to stop using something. |
I'm closing this since we moved the docs to their respective projects and https://github.com/newrelic/micrometer-registry-newrelic is deprecated wile Micrometer's New Relic registry is still supported. |
Hi. We'd like to see about updating these older docs to more proactively point to our dimensional metrics based registry. Would love to know what you think and what else we can do to help get these changed.
Thanks!