Add instrumentation for fs.glob for node 22+
#2336
Assignees
Comments
bizob2828
moved this to Triage Needed: Unprioritized Features
in Node.js Engineering Board
jsumners-nr
moved this from Triage Needed: Unprioritized Features
to In progress: Issues being worked on
in Node.js Engineering Board
github-project-automation
bot
moved this from In progress: Issues being worked on
to Done: Issues recently completed
in Node.js Engineering Board
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
In #2304 I identified a useful method to add to our fs instrumentation. We purposely avoid instrumenting the sync methods because they are used in tight loops in other parts of core and frameworks so you’d get a pile of overhead and thousands of segment you don’t want (could potentially merge them but overhead maybe not worth it)
Acceptance Criteria
Add instrumentation for
fs.globin Node 22+Design Consideration/Limitations
Why is this the route we should take to achieve our goal?
What can't be achieved within this story?
Dependencies
Do any other teams or parts of the New Relic product need to be considered?
Some common areas: UI, collector, documentation
Additional context
We need to add an integration test for this new method. Looking at the tests we could reduce a lot of boiler plate too, but up to the dev that takes this on
The text was updated successfully, but these errors were encountered: