Skip to content

Conversation

rgaignault
Copy link
Contributor

@rgaignault rgaignault commented Oct 1, 2025

Motivation

After testing the graphql tracking on datadog internal app we can now remove the feature flag.

Changes

Test instructions

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.

@rgaignault rgaignault requested a review from a team as a code owner October 1, 2025 09:07
Copy link

cit-pr-commenter bot commented Oct 1, 2025

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 161.81 KiB 161.72 KiB -85 B -0.05%
Rum Recorder 19.43 KiB 19.43 KiB 0 B 0.00%
Rum Profiler 5.32 KiB 5.32 KiB 0 B 0.00%
Logs 56.06 KiB 56.02 KiB -38 B -0.07%
Flagging 944 B 944 B 0 B 0.00%
Rum Slim 119.44 KiB 119.37 KiB -77 B -0.06%
Worker 23.60 KiB 23.60 KiB 0 B 0.00%
🚀 CPU Performance
Action Name Base CPU Time (ms) Local CPU Time (ms) 𝚫%
RUM - add global context 0.0078 0.0042 -46.15%
RUM - add action 0.0266 0.0126 -52.63%
RUM - add error 0.0211 0.0119 -43.60%
RUM - add timing 0.0054 0.0027 -50.00%
RUM - start view 0.0069 0.0033 -52.17%
RUM - start/stop session replay recording 0.0013 0.0007 -46.15%
Logs - log message 0.0259 0.014 -45.95%
🧠 Memory Performance
Action Name Base Memory Consumption Local Memory Consumption 𝚫
RUM - add global context 25.64 KiB 25.91 KiB +275 B
RUM - add action 43.05 KiB 45.76 KiB +2.71 KiB
RUM - add timing 25.72 KiB 24.75 KiB -987 B
RUM - add error 48.81 KiB 51.84 KiB +3.02 KiB
RUM - start/stop session replay recording 24.66 KiB 23.92 KiB -766 B
RUM - start view 421.80 KiB 430.72 KiB +8.92 KiB
Logs - log message 41.64 KiB 43.59 KiB +1.95 KiB

🔗 RealWorld

@datadog-datadog-prod-us1
Copy link

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 92.50% (-0.00%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: cfe7f89 | Docs | Was this helpful? Give us feedback!

@rgaignault rgaignault merged commit b220edf into main Oct 2, 2025
21 checks passed
@rgaignault rgaignault deleted the romanG/remove-graphql-tracking-ff branch October 2, 2025 14:01
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