designcourse icon indicating copy to clipboard operation
designcourse copied to clipboard

⚡️ Remove customer data track for perf and bundle size

Open cy-moi opened this issue 1 year ago • 2 comments

Motivation

Customer data tracker can cause performance overhead and we want to decrease bundle size.

Changes

Remove customer data tracker.

Testing

  • [x] Local
  • [ ] Staging
  • [x] Unit
  • [ ] End to end

I have gone over the contributing documentation.

cy-moi avatar Mar 05 '25 09:03 cy-moi

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.90%. Comparing base (56030c6) to head (bd8be5d).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3393      +/-   ##
==========================================
- Coverage   91.94%   91.90%   -0.04%     
==========================================
  Files         312      311       -1     
  Lines        8122     8034      -88     
  Branches     1837     1819      -18     
==========================================
- Hits         7468     7384      -84     
+ Misses        654      650       -4     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Mar 21 '25 13:03 codecov-commenter

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 147.43 KiB 145.35 KiB -2131 B -1.41%
Logs 51.40 KiB 50.16 KiB -1264 B -2.40%
Rum Slim 106.79 KiB 104.80 KiB -2040 B -1.87%
Worker 23.59 KiB 23.59 KiB 0 B 0.00%
🚀 CPU Performance
Action Name Base Average Cpu Time (ms) Local Average Cpu Time (ms) 𝚫
addglobalcontext 0.003 0.003 0.001
addaction 0.044 0.070 0.027
addtiming 0.002 0.002 0.001
adderror 0.045 0.076 0.031
startstopsessionreplayrecording 0.010 0.033 0.023
startview 0.431 0.615 0.184
logmessage 0.024 0.043 0.019
🧠 Memory Performance
Action Name Base Consumption Memory (bytes) Local Consumption Memory (bytes) 𝚫 (bytes)
addglobalcontext 30.18 KiB 31.13 KiB 973 B
addaction 56.95 KiB 58.78 KiB 1.83 KiB
addtiming 26.89 KiB 26.78 KiB -116 B
adderror 60.07 KiB 61.84 KiB 1.76 KiB
startstopsessionreplayrecording 26.04 KiB 25.64 KiB -412 B
startview 428.28 KiB 428.36 KiB 78 B
logmessage 55.17 KiB 56.90 KiB 1.74 KiB

🔗 RealWorld

cit-pr-commenter[bot] avatar Mar 21 '25 13:03 cit-pr-commenter[bot]

/to-staging

cy-moi avatar Apr 01 '25 13:04 cy-moi

View all feedbacks in Devflow UI. 2025-04-01 13:46:09 UTC :information_source: Start processing command /to-staging


2025-04-01 13:46:20 UTC :information_source: Branch Integration: starting soon, merge expected in approximately 15m44s (p90)

Commit bd8be5d4ad will soon be integrated into staging-14.


2025-04-01 13:59:12 UTC :information_source: Branch Integration: This commit was successfully integrated

Commit bd8be5d4ad has been merged into staging-14 in merge commit 7e96e9fac3.

Check out the triggered pipeline on Gitlab :fox_face:

If you need to revert this integration, you can use the following command: /code revert-integration -b staging-14

dd-devflow[bot] avatar Apr 01 '25 13:04 dd-devflow[bot]