Skip to content

RUM-16284: Cache lastExitConsoleLog in RumCrashReporterTask#153

Merged
kikoveiga merged 1 commit into
developfrom
kikoveiga/RUM-16284/cache-crash-reporter-fields
May 29, 2026
Merged

RUM-16284: Cache lastExitConsoleLog in RumCrashReporterTask#153
kikoveiga merged 1 commit into
developfrom
kikoveiga/RUM-16284/cache-crash-reporter-fields

Conversation

@kikoveiga

@kikoveiga kikoveiga commented May 14, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

  • Cache lastExitConsoleLog in RumCrashReporterTask to reduce rendezvous.
  • Reuse lastViewEvent.context, reducing rendezvous occurrences and aligning the behavior with the other SDKs.
  • Change ddLogWarning to ddLogVerbose to reduce warnings noise.

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)

@kikoveiga kikoveiga self-assigned this May 15, 2026
@kikoveiga
kikoveiga force-pushed the kikoveiga/RUM-16284/cache-crash-reporter-fields branch 2 times, most recently from 9c6ddfb to 5e86eb6 Compare May 22, 2026 14:31
@datadog-datadog-prod-us1-2

This comment has been minimized.

@kikoveiga
kikoveiga force-pushed the kikoveiga/RUM-16284/cache-crash-reporter-fields branch from 5e86eb6 to ed44db9 Compare May 22, 2026 14:55
@kikoveiga kikoveiga changed the title RUM-16284: Cache lastExitConsoleLog and datadogContext in RumCrashReporterTask RUM-16284: Cache lastExitConsoleLog in RumCrashReporterTask May 22, 2026
@kikoveiga
kikoveiga marked this pull request as ready for review May 22, 2026 15:00
@kikoveiga
kikoveiga requested a review from a team as a code owner May 22, 2026 15:00
@kikoveiga
kikoveiga merged commit 51ef1fe into develop May 29, 2026
8 checks passed
@kikoveiga
kikoveiga deleted the kikoveiga/RUM-16284/cache-crash-reporter-fields branch May 29, 2026 08:55
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.

2 participants