Skip to content

Prefer immutable set for test initialization #4904

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 17, 2023
Merged

Prefer immutable set for test initialization #4904

merged 2 commits into from
Apr 17, 2023

Conversation

rlazo
Copy link
Collaborator

@rlazo rlazo commented Apr 17, 2023

No description provided.

@rlazo rlazo requested a review from VinayGuthal April 17, 2023 20:07
@rlazo rlazo requested a review from daymxn April 17, 2023 20:07
@github-actions
Copy link
Contributor

github-actions bot commented Apr 17, 2023

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

@google-oss-bot
Copy link
Contributor

1 Warning
⚠️ Did you forget to add a changelog entry? (Add the 'no-changelog' label to the PR to silence this warning.)

Generated by 🚫 Danger

@google-oss-bot
Copy link
Contributor

@rlazo rlazo enabled auto-merge (squash) April 17, 2023 20:13
@github-actions
Copy link
Contributor

Unit Test Results

  34 files   -    705    34 suites   - 705   1m 28s ⏱️ - 30m 20s
  57 tests  - 4 807    57 ✔️  - 4 783  0 💤  - 21  0  - 3 
114 runs   - 8 742  114 ✔️  - 8 698  0 💤  - 41  0  - 3 

Results for commit a105cee. ± Comparison against base commit 8840eee.

Copy link
Member

@daymxn daymxn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This gives me an idea...

@rlazo rlazo merged commit 7e49209 into master Apr 17, 2023
@rlazo rlazo deleted the tem branch April 17, 2023 20:21
@google-oss-bot
Copy link
Contributor

@google-oss-bot
Copy link
Contributor

Startup Time Report 1

Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS.

Notes

Startup Times

  • ComponentDiscovery

    DeviceStatisticsDistributions
    oriole-32
    Percentile8840eee578faf2DiffSignificant (?)
    p10460 ±53 μs443 ±26 μs-16.6 μs (-3.6%)NO
    p25504 ±62 μs489 ±30 μs-15.0 μs (-3.0%)NO
    p50606 ±66 μs580 ±34 μs-26.7 μs (-4.4%)NO
    p75726 ±88 μs689 ±43 μs-36.6 μs (-5.0%)NO
    p90893 ±103 μs860 ±52 μs-32.7 μs (-3.7%)NO

    20 test runs in comparison
    CommitTest Runs
    8840eee
    • 2023-04-17_17:40:38.170833_tmml
    • 2023-04-17_17:40:38.175495_Gudd
    • 2023-04-17_17:40:38.175516_vHCe
    • 2023-04-17_17:40:38.175524_tLlG
    • 2023-04-17_17:40:38.175534_YUxK
    • 2023-04-17_17:40:38.175549_SNba
    • 2023-04-17_17:40:38.175567_Meym
    • 2023-04-17_17:40:38.175574_TlhS
    • 2023-04-17_17:40:38.175580_LIVi
    • 2023-04-17_17:40:38.175586_TtcW
    578faf2
    • 2023-04-17_20:29:05.629768_xObz
    • 2023-04-17_20:29:05.635317_Lpyk
    • 2023-04-17_20:29:05.635336_Cvgt
    • 2023-04-17_20:29:05.635344_XWyG
    • 2023-04-17_20:29:05.635351_zqBK
    • 2023-04-17_20:29:05.635365_pltd
    • 2023-04-17_20:29:05.635372_Dvmz
    • 2023-04-17_20:29:05.635379_szzk
    • 2023-04-17_20:29:05.635384_rydo
    • 2023-04-17_20:29:05.635391_XBsi
    redfin-30
    Percentile8840eee578faf2DiffSignificant (?)
    p10766 ±18 μs761 ±14 μs-4.93 μs (-0.6%)NO
    p25800 ±19 μs800 ±18 μs-595 ns (-0.1%)NO
    p50862 ±21 μs881 ±45 μs+19.0 μs (+2.2%)NO
    p751.06 ±0.08 ms1.13 ±0.1 ms+65.5 μs (+6.2%)NO
    p901.55 ±0.2 ms1.60 ±0.3 ms+51.5 μs (+3.3%)NO

    20 test runs in comparison
    CommitTest Runs
    8840eee
    • 2023-04-17_17:40:38.170833_tmml
    • 2023-04-17_17:40:38.175495_Gudd
    • 2023-04-17_17:40:38.175516_vHCe
    • 2023-04-17_17:40:38.175524_tLlG
    • 2023-04-17_17:40:38.175534_YUxK
    • 2023-04-17_17:40:38.175549_SNba
    • 2023-04-17_17:40:38.175567_Meym
    • 2023-04-17_17:40:38.175574_TlhS
    • 2023-04-17_17:40:38.175580_LIVi
    • 2023-04-17_17:40:38.175586_TtcW
    578faf2
    • 2023-04-17_20:29:05.629768_xObz
    • 2023-04-17_20:29:05.635317_Lpyk
    • 2023-04-17_20:29:05.635336_Cvgt
    • 2023-04-17_20:29:05.635344_XWyG
    • 2023-04-17_20:29:05.635351_zqBK
    • 2023-04-17_20:29:05.635365_pltd
    • 2023-04-17_20:29:05.635372_Dvmz
    • 2023-04-17_20:29:05.635379_szzk
    • 2023-04-17_20:29:05.635384_rydo
    • 2023-04-17_20:29:05.635391_XBsi
  • Firebase

    DeviceStatisticsDistributions
    oriole-32
    Percentile8840eee578faf2DiffSignificant (?)
    p103.96 ±0.5 ms3.87 ±0.2 ms-96.6 μs (-2.4%)NO
    p254.31 ±0.8 ms4.06 ±0.3 ms-248 μs (-5.8%)NO
    p504.69 ±1 ms4.41 ±0.3 ms-285 μs (-6.1%)NO
    p755.22 ±1 ms4.94 ±0.4 ms-275 μs (-5.3%)NO
    p905.93 ±1 ms5.63 ±0.4 ms-300 μs (-5.1%)NO

    20 test runs in comparison
    CommitTest Runs
    8840eee
    • 2023-04-17_17:40:38.170833_tmml
    • 2023-04-17_17:40:38.175495_Gudd
    • 2023-04-17_17:40:38.175516_vHCe
    • 2023-04-17_17:40:38.175524_tLlG
    • 2023-04-17_17:40:38.175534_YUxK
    • 2023-04-17_17:40:38.175549_SNba
    • 2023-04-17_17:40:38.175567_Meym
    • 2023-04-17_17:40:38.175574_TlhS
    • 2023-04-17_17:40:38.175580_LIVi
    • 2023-04-17_17:40:38.175586_TtcW
    578faf2
    • 2023-04-17_20:29:05.629768_xObz
    • 2023-04-17_20:29:05.635317_Lpyk
    • 2023-04-17_20:29:05.635336_Cvgt
    • 2023-04-17_20:29:05.635344_XWyG
    • 2023-04-17_20:29:05.635351_zqBK
    • 2023-04-17_20:29:05.635365_pltd
    • 2023-04-17_20:29:05.635372_Dvmz
    • 2023-04-17_20:29:05.635379_szzk
    • 2023-04-17_20:29:05.635384_rydo
    • 2023-04-17_20:29:05.635391_XBsi
    redfin-30
    Percentile8840eee578faf2DiffSignificant (?)
    p106.57 ±0.2 ms6.53 ±0.1 ms-36.5 μs (-0.6%)NO
    p256.75 ±0.2 ms6.72 ±0.1 ms-21.0 μs (-0.3%)NO
    p507.08 ±0.2 ms7.11 ±0.2 ms+33.7 μs (+0.5%)NO
    p757.81 ±0.2 ms7.81 ±0.2 ms-2.36 μs (-0.0%)NO
    p909.02 ±0.5 ms9.11 ±0.6 ms+86.1 μs (+1.0%)NO

    20 test runs in comparison
    CommitTest Runs
    8840eee
    • 2023-04-17_17:40:38.170833_tmml
    • 2023-04-17_17:40:38.175495_Gudd
    • 2023-04-17_17:40:38.175516_vHCe
    • 2023-04-17_17:40:38.175524_tLlG
    • 2023-04-17_17:40:38.175534_YUxK
    • 2023-04-17_17:40:38.175549_SNba
    • 2023-04-17_17:40:38.175567_Meym
    • 2023-04-17_17:40:38.175574_TlhS
    • 2023-04-17_17:40:38.175580_LIVi
    • 2023-04-17_17:40:38.175586_TtcW
    578faf2
    • 2023-04-17_20:29:05.629768_xObz
    • 2023-04-17_20:29:05.635317_Lpyk
    • 2023-04-17_20:29:05.635336_Cvgt
    • 2023-04-17_20:29:05.635344_XWyG
    • 2023-04-17_20:29:05.635351_zqBK
    • 2023-04-17_20:29:05.635365_pltd
    • 2023-04-17_20:29:05.635372_Dvmz
    • 2023-04-17_20:29:05.635379_szzk
    • 2023-04-17_20:29:05.635384_rydo
    • 2023-04-17_20:29:05.635391_XBsi
  • Runtime

    DeviceStatisticsDistributions
    oriole-32
    Percentile8840eee578faf2DiffSignificant (?)
    p103.39 ±0.4 ms3.32 ±0.2 ms-61.2 μs (-1.8%)NO
    p253.69 ±0.7 ms3.49 ±0.2 ms-202 μs (-5.5%)NO
    p504.00 ±0.9 ms3.75 ±0.3 ms-246 μs (-6.2%)NO
    p754.46 ±0.9 ms4.18 ±0.3 ms-280 μs (-6.3%)NO
    p905.10 ±1 ms4.84 ±0.4 ms-260 μs (-5.1%)NO

    20 test runs in comparison
    CommitTest Runs
    8840eee
    • 2023-04-17_17:40:38.170833_tmml
    • 2023-04-17_17:40:38.175495_Gudd
    • 2023-04-17_17:40:38.175516_vHCe
    • 2023-04-17_17:40:38.175524_tLlG
    • 2023-04-17_17:40:38.175534_YUxK
    • 2023-04-17_17:40:38.175549_SNba
    • 2023-04-17_17:40:38.175567_Meym
    • 2023-04-17_17:40:38.175574_TlhS
    • 2023-04-17_17:40:38.175580_LIVi
    • 2023-04-17_17:40:38.175586_TtcW
    578faf2
    • 2023-04-17_20:29:05.629768_xObz
    • 2023-04-17_20:29:05.635317_Lpyk
    • 2023-04-17_20:29:05.635336_Cvgt
    • 2023-04-17_20:29:05.635344_XWyG
    • 2023-04-17_20:29:05.635351_zqBK
    • 2023-04-17_20:29:05.635365_pltd
    • 2023-04-17_20:29:05.635372_Dvmz
    • 2023-04-17_20:29:05.635379_szzk
    • 2023-04-17_20:29:05.635384_rydo
    • 2023-04-17_20:29:05.635391_XBsi
    redfin-30
    Percentile8840eee578faf2DiffSignificant (?)
    p105.63 ±0.2 ms5.61 ±0.1 ms-28.2 μs (-0.5%)NO
    p255.79 ±0.2 ms5.77 ±0.1 ms-16.5 μs (-0.3%)NO
    p506.05 ±0.2 ms6.03 ±0.1 ms-26.8 μs (-0.4%)NO
    p756.52 ±0.2 ms6.47 ±0.2 ms-45.7 μs (-0.7%)NO
    p907.29 ±0.4 ms7.27 ±0.3 ms-21.5 μs (-0.3%)NO

    20 test runs in comparison
    CommitTest Runs
    8840eee
    • 2023-04-17_17:40:38.170833_tmml
    • 2023-04-17_17:40:38.175495_Gudd
    • 2023-04-17_17:40:38.175516_vHCe
    • 2023-04-17_17:40:38.175524_tLlG
    • 2023-04-17_17:40:38.175534_YUxK
    • 2023-04-17_17:40:38.175549_SNba
    • 2023-04-17_17:40:38.175567_Meym
    • 2023-04-17_17:40:38.175574_TlhS
    • 2023-04-17_17:40:38.175580_LIVi
    • 2023-04-17_17:40:38.175586_TtcW
    578faf2
    • 2023-04-17_20:29:05.629768_xObz
    • 2023-04-17_20:29:05.635317_Lpyk
    • 2023-04-17_20:29:05.635336_Cvgt
    • 2023-04-17_20:29:05.635344_XWyG
    • 2023-04-17_20:29:05.635351_zqBK
    • 2023-04-17_20:29:05.635365_pltd
    • 2023-04-17_20:29:05.635372_Dvmz
    • 2023-04-17_20:29:05.635379_szzk
    • 2023-04-17_20:29:05.635384_rydo
    • 2023-04-17_20:29:05.635391_XBsi
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentile8840eee578faf2DiffSignificant (?)
    p10198 ±17 ms201 ±7 ms+3.18 ms (+1.6%)NO
    p25208 ±23 ms207 ±8 ms-265 μs (-0.1%)NO
    p50220 ±33 ms216 ±8 ms-4.27 ms (-1.9%)NO
    p75230 ±38 ms226 ±9 ms-4.21 ms (-1.8%)NO
    p90240 ±41 ms237 ±11 ms-2.49 ms (-1.0%)NO

    20 test runs in comparison
    CommitTest Runs
    8840eee
    • 2023-04-17_17:40:38.170833_tmml
    • 2023-04-17_17:40:38.175495_Gudd
    • 2023-04-17_17:40:38.175516_vHCe
    • 2023-04-17_17:40:38.175524_tLlG
    • 2023-04-17_17:40:38.175534_YUxK
    • 2023-04-17_17:40:38.175549_SNba
    • 2023-04-17_17:40:38.175567_Meym
    • 2023-04-17_17:40:38.175574_TlhS
    • 2023-04-17_17:40:38.175580_LIVi
    • 2023-04-17_17:40:38.175586_TtcW
    578faf2
    • 2023-04-17_20:29:05.629768_xObz
    • 2023-04-17_20:29:05.635317_Lpyk
    • 2023-04-17_20:29:05.635336_Cvgt
    • 2023-04-17_20:29:05.635344_XWyG
    • 2023-04-17_20:29:05.635351_zqBK
    • 2023-04-17_20:29:05.635365_pltd
    • 2023-04-17_20:29:05.635372_Dvmz
    • 2023-04-17_20:29:05.635379_szzk
    • 2023-04-17_20:29:05.635384_rydo
    • 2023-04-17_20:29:05.635391_XBsi
    redfin-30
    Percentile8840eee578faf2DiffSignificant (?)
    p10231 ±3 ms252 ±4 ms+21.3 ms (+9.2%)YES
    p25237 ±4 ms258 ±4 ms+20.7 ms (+8.7%)YES
    p50244 ±4 ms265 ±5 ms+21.3 ms (+8.7%)MAYBE
    p75253 ±6 ms273 ±6 ms+20.3 ms (+8.0%)MAYBE
    p90261 ±6 ms282 ±6 ms+21.6 ms (+8.3%)MAYBE

    20 test runs in comparison
    CommitTest Runs
    8840eee
    • 2023-04-17_17:40:38.170833_tmml
    • 2023-04-17_17:40:38.175495_Gudd
    • 2023-04-17_17:40:38.175516_vHCe
    • 2023-04-17_17:40:38.175524_tLlG
    • 2023-04-17_17:40:38.175534_YUxK
    • 2023-04-17_17:40:38.175549_SNba
    • 2023-04-17_17:40:38.175567_Meym
    • 2023-04-17_17:40:38.175574_TlhS
    • 2023-04-17_17:40:38.175580_LIVi
    • 2023-04-17_17:40:38.175586_TtcW
    578faf2
    • 2023-04-17_20:29:05.629768_xObz
    • 2023-04-17_20:29:05.635317_Lpyk
    • 2023-04-17_20:29:05.635336_Cvgt
    • 2023-04-17_20:29:05.635344_XWyG
    • 2023-04-17_20:29:05.635351_zqBK
    • 2023-04-17_20:29:05.635365_pltd
    • 2023-04-17_20:29:05.635372_Dvmz
    • 2023-04-17_20:29:05.635379_szzk
    • 2023-04-17_20:29:05.635384_rydo
    • 2023-04-17_20:29:05.635391_XBsi

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/BixnJRirs3/index.html

@firebase firebase locked and limited conversation to collaborators May 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants