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

Update New Relic docs #137

Closed
wants to merge 21 commits into from

Conversation

breedx-nr
Copy link

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!

@breedx-nr
Copy link
Author

Just wanted to leave a gentle poke to see if anybody might be able to look at this. Thanks.

@breedx-nr
Copy link
Author

Bumping again. Would it be possible to get an estimate on when we might get eyes on this? 🙏 🙇‍♂️

@breedx-nr
Copy link
Author

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
Copy link

@ebullient ebullient Aug 12, 2020

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. ;)

@ebullient
Copy link

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?)

@breedx-nr
Copy link
Author

breedx-nr commented Aug 12, 2020

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.

@shakuzen
Copy link
Member

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.

@jonatan-ivanov
Copy link
Member

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants