[IGNORE] Community PR 5961#5976
Conversation
Strech
commented
Jul 1, 2026
Inject and extract the DSM pathway context through the serialized job payload so produce (enqueue) and consume (perform) checkpoints connect across the enqueue/perform process boundary, regardless of queue adapter.
Require datadog/data_streams so the enabled? facade is defined under the tracing-only entrypoint, avoiding a NoMethodError on every serialize. Collapse the two single-method patches into one DataStreams module and add a regression test for the unconfigured-DSM path.
|
👋 Hey @Strech, please fill "Change log entry" section in the pull request description. If changes need to be present in CHANGELOG.md you can state it this way **Change log entry**
Yes. A brief summary to be placed into the CHANGELOG.md(possible answers Yes/Yep/Yeah) Or you can opt out like that **Change log entry**
None.(possible answers No/Nope/None) Visited at: 2026-07-01 14:06:35 UTC |
Typing analysisThis PR does not change typing compared to the base branch. |
|
BenchmarksBenchmark execution time: 2026-07-03 09:36:39 Comparing candidate commit 251ce66 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 48 metrics, 1 unstable metrics.
|
ActiveJob 4.2 deserializes jobs via a class method, not the instance method, so a prepended instance deserialize has no super to call and cannot receive the queue. Gate DSM injection to ActiveJob 5.0+ and skip the DSM specs below that version. Replace untyped with any in the RBS and add a module self-type so steep can resolve queue_name on the host job class.
98caaf5 to
254fd79
Compare
16af2bf to
251ce66
Compare