-
Notifications
You must be signed in to change notification settings - Fork 360
feat(tracing): add bullmq integration and dsm support #7123
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
base: master
Are you sure you want to change the base?
Conversation
Overall package sizeSelf size: 4.38 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.0 | 68.46 kB | 797.03 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7123 +/- ##
===========================================
- Coverage 84.54% 69.78% -14.76%
===========================================
Files 529 285 -244
Lines 22542 10505 -12037
===========================================
- Hits 19058 7331 -11727
+ Misses 3484 3174 -310 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
BenchmarksBenchmark execution time: 2026-01-08 17:33:10 Comparing candidate commit 810ed85 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 292 metrics, 28 unstable metrics. |
packages/datadog-instrumentations/src/helpers/rewriter/instrumentations/index.js
Show resolved
Hide resolved
|
Shouldn't there be a new |
@tlhunter What is this? I'm not aware of what this is |
This comment has been minimized.
This comment has been minimized.
I was saying we need a new test to appear in the list of github actions that have executed as a result of this pull request. Such a test was not present when I reviewed previously but is now working.
|
| return { | ||
| queueName: ctx.self?.name || 'bullmq', | ||
| payloadSize: data ? getMessageSize(data) : 0, | ||
| injectTarget: data |
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.
Does BullMQ just stick JSON strings into Redis and we're adding a new field to it for context prop?
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 basically (as well as allowing just about any type)
Co-authored-by: Thomas Hunter II <tlhunter@datadog.com>
Co-authored-by: Thomas Hunter II <tlhunter@datadog.com>
Co-authored-by: Thomas Hunter II <tlhunter@datadog.com>
c657641 to
aef1d02
Compare
|
@tlhunter The orchestrion configs look fine. |

What does this PR do?
This PR was auto-generated by the APM AI Integration Toolkit and adds APM Tracing for the bullmq package.
Summary of Changes
Motivation
Plugin Checklist
Additional Notes