search

newrelic / node-newrelic

New Relic Node.js agent code base. Developers are welcome to create pull requests here, please see our contributing guidelines. For New Relic technical support, please go to http://support.newrelic.com.
Apache License 2.0
968 stars 398 forks source link

Add Support for SQL Explain Plans #2257

Open christhekeele opened 4 months ago

christhekeele commented 4 months ago

Feature Description

A clear and concise description of the feature you want or need.

Like many other agents do today, it would be helpful if New Relic auto-explained slow query traces (a functionality I understand is referred to as "SQL explain plans") in the Node agent as well.

Additional context

Support thread prompting this feature request here: https://forum.newrelic.com/s/hubtopic/aAXPh00000022wzOAA/unable-to-view-sql-explain-plans-for-slow-queries-in-new-relic

Priority

Please help us better understand this feature request by choosing a priority from the following options: [Nice to Have, Really Want, Must Have, Blocker]

[Must Have] for me, I was really surprised to learn the Node agent does not have feature parity here with other languages, as slow queries are THE most common performance bottleneck in web applications; and understanding why they are slow seems universally useful. For example, knowing this feature is not available I don't know if I would choose NR when standing up future Node projects, whereas I still would for ex. a python project. Pretty table stakes stuff.

Describe Alternatives

A clear and concise description of any alternative solutions or features you've considered. Are there examples you could link us to?

Not really. Switching backend languages or switching off of new relic? I looked into implementing this myself as a custom Datastore Implementation but the example was too anemic for me to successfully get my own basic datastore off the ground, let alone support more advanced features. It feels like if any builtin datastore implementation should have all the NR features out of the box, it should be SQL, but reimplementing all of it for this one feature is a large maintenance burden for any project.

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

As a user, I'm always frustrated when I have to write my own user stories. 😉

workato-integration[bot] commented 4 months ago

https://new-relic.atlassian.net/browse/NR-278066

bizob2828 commented 4 months ago

Hi @christhekeele thanks for the feature request. We will be doing some investigation on our end and prioritize this request.