Skip to content

RUM-15323: RUM sends ANR & LongTask events to Profiling Feature#3301

Merged
ambushwork merged 1 commit intofeature/continuous-profilingfrom
yl/profiling/rum-events-to-continuous-profiling
Mar 30, 2026
Merged

RUM-15323: RUM sends ANR & LongTask events to Profiling Feature#3301
ambushwork merged 1 commit intofeature/continuous-profilingfrom
yl/profiling/rum-events-to-continuous-profiling

Conversation

@ambushwork
Copy link
Copy Markdown
Member

What does this PR do?

This PR implements the logic of sending RUM long-task and ANR events to the Profiling feature for continuous profiling correlation.

It also adds the profiling status for these two RUM events.

Motivation

RUM-15323

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@ambushwork ambushwork force-pushed the yl/profiling/rum-events-to-continuous-profiling branch 2 times, most recently from 57d30a7 to ab5cc96 Compare March 27, 2026 13:52
@ambushwork
Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ambushwork ambushwork marked this pull request as ready for review March 27, 2026 14:17
@ambushwork ambushwork requested review from a team as code owners March 27, 2026 14:17
@datadog-datadog-prod-us1

This comment has been minimized.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 27, 2026

Codecov Report

❌ Patch coverage is 69.09091% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.54%. Comparing base (f5e2767) to head (ed4a6d2).

Files with missing lines Patch % Lines
...nternal/profiling/ContinuousProfilingRumContext.kt 0.00% 5 Missing ⚠️
.../datadog/android/internal/profiling/RumAnrEvent.kt 0.00% 4 Missing ⚠️
...dog/android/internal/profiling/RumLongTaskEvent.kt 0.00% 4 Missing ⚠️
...dog/android/profiling/internal/ProfilingFeature.kt 0.00% 0 Missing and 2 partials ⚠️
.../android/rum/internal/domain/scope/RumViewScope.kt 95.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@                       Coverage Diff                        @@
##           feature/continuous-profiling    #3301      +/-   ##
================================================================
+ Coverage                         71.47%   71.54%   +0.07%     
================================================================
  Files                               944      947       +3     
  Lines                             34873    34926      +53     
  Branches                           5912     5921       +9     
================================================================
+ Hits                              24924    24985      +61     
+ Misses                             8285     8279       -6     
+ Partials                           1664     1662       -2     
Files with missing lines Coverage Δ
...og/android/rum/internal/anr/ANRDetectorRunnable.kt 67.61% <ø> (ø)
...dog/android/profiling/internal/ProfilingFeature.kt 63.79% <0.00%> (-2.28%) ⬇️
.../android/rum/internal/domain/scope/RumViewScope.kt 94.26% <95.00%> (+0.43%) ⬆️
.../datadog/android/internal/profiling/RumAnrEvent.kt 0.00% <0.00%> (ø)
...dog/android/internal/profiling/RumLongTaskEvent.kt 0.00% <0.00%> (ø)
...nternal/profiling/ContinuousProfilingRumContext.kt 0.00% <0.00%> (ø)

... and 32 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ambushwork ambushwork force-pushed the yl/profiling/rum-events-to-continuous-profiling branch from ab5cc96 to ad48ed2 Compare March 30, 2026 08:31
@ambushwork ambushwork requested a review from 0xnm March 30, 2026 08:32
@ambushwork ambushwork force-pushed the yl/profiling/rum-events-to-continuous-profiling branch from ad48ed2 to 081fab4 Compare March 30, 2026 09:00
@ambushwork ambushwork force-pushed the yl/profiling/rum-events-to-continuous-profiling branch from 081fab4 to 6c74110 Compare March 30, 2026 12:51
@ambushwork ambushwork force-pushed the yl/profiling/rum-events-to-continuous-profiling branch from 6c74110 to ed4a6d2 Compare March 30, 2026 13:25
@ambushwork ambushwork requested a review from 0xnm March 30, 2026 14:18
@ambushwork ambushwork merged commit 64e4177 into feature/continuous-profiling Mar 30, 2026
26 checks passed
@ambushwork ambushwork deleted the yl/profiling/rum-events-to-continuous-profiling branch March 30, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants