Skip to content

Add different default sampling rates for when RC fetch failed #5059

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 5 commits into from
Jun 7, 2023

Conversation

mrober
Copy link
Contributor

@mrober mrober commented Jun 7, 2023

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 7, 2023

Release note changes

The following release notes were modified. Please ensure they look correct.

Release Notes
firebase-perf
### {{perfmon}} version 20.3.3 {: #performance_v20-3-3}

* {{fixed}} Fixed app start trace creation where some measured time could be NULL GitHub [#4730](//github.com/firebase/firebase-android-sdk/issues/4730){: .external}.

* {{changed}} Adjusted default behavior when remote config fetch fails.

#### {{perfmon}} Kotlin extensions version 20.3.3 {: #performance-ktx_v20-3-3}

The Kotlin extensions library transitively includes the updated
`firebase-performance` library. The Kotlin extensions library has no additional
updates.

@mrober mrober requested a review from visumickey June 7, 2023 13:33
@github-actions
Copy link
Contributor

github-actions bot commented Jun 7, 2023

Unit Test Results

   102 files  ±  0     102 suites  ±0   2m 49s ⏱️ +14s
   981 tests +  6     981 ✔️ +  6  0 💤 ±0  0 ±0 
1 962 runs  +12  1 962 ✔️ +12  0 💤 ±0  0 ±0 

Results for commit d081ae3. ± Comparison against base commit 174e7d8.

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 7, 2023

Coverage Report 1

Affected Products

  • firebase-perf

    Overall coverage changed from 71.19% (174e7d8) to 71.25% (b7d3504) by +0.06%.

    FilenameBase (174e7d8)Merge (b7d3504)Diff
    ConfigResolver.java93.19%93.43%+0.24%
    ConfigurationConstants.java97.86%97.93%+0.07%
    ConfigurationFlag.java100.00%80.00%-20.00%
    RemoteConfigManager.java91.30%91.43%+0.12%

Test Logs

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

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 7, 2023

Size Report 1

Affected Products

  • firebase-perf

    TypeBase (174e7d8)Merge (b7d3504)Diff
    aar315 kB316 kB+685 B (+0.2%)
    apk (aggressive)1.06 MB1.06 MB+480 B (+0.0%)
    apk (release)3.06 MB3.06 MB+340 B (+0.0%)

Test Logs

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

Copy link
Contributor

@visumickey visumickey left a comment

Choose a reason for hiding this comment

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

Changes look good to me.

I'm just thinking if the CPU and memory capturing frequency can also be reduced incase of RC fetch failures. Example: instead of capturing every 100ms, we could push the limits to once in every 300ms(??).

@visumickey visumickey requested a review from jposuna June 7, 2023 13:56
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 7, 2023

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

  • fire-perf

    DeviceStatisticsDistributions
    oriole-32
    Percentile174e7d8b7d3504DiffSignificant (?)
    p10263 ±21 μs248 ±20 μs-15.3 μs (-5.8%)NO
    p25275 ±23 μs258 ±18 μs-17.0 μs (-6.2%)NO
    p50291 ±22 μs276 ±18 μs-15.5 μs (-5.3%)NO
    p75321 ±21 μs305 ±16 μs-16.3 μs (-5.1%)NO
    p90388 ±35 μs359 ±18 μs-29.6 μs (-7.6%)NO

    20 test runs in comparison
    CommitTest Runs
    174e7d8
    • 2023-06-06_19:40:46.437159_kiPG
    • 2023-06-06_19:40:46.441888_ZaRR
    • 2023-06-06_19:40:46.442049_taQy
    • 2023-06-06_19:40:46.442059_mQMZ
    • 2023-06-06_19:40:46.442065_XgyR
    • 2023-06-06_19:40:46.442071_tgtS
    • 2023-06-06_19:40:46.442076_kGKy
    • 2023-06-06_19:40:46.442090_qaPF
    • 2023-06-06_19:40:46.442096_JNez
    • 2023-06-06_19:40:46.442102_eszK
    b7d3504
    • 2023-06-07_15:02:08.787186_wzlv
    • 2023-06-07_15:02:08.793483_bHia
    • 2023-06-07_15:02:08.793506_wFHy
    • 2023-06-07_15:02:08.793514_kise
    • 2023-06-07_15:02:08.793522_CrOP
    • 2023-06-07_15:02:08.793530_SkgK
    • 2023-06-07_15:02:08.793538_MkYO
    • 2023-06-07_15:02:08.793544_DOAe
    • 2023-06-07_15:02:08.793552_ohjc
    • 2023-06-07_15:02:08.793559_soqF
    redfin-30
    Percentile174e7d8b7d3504DiffSignificant (?)
    p10794 ±51 μs748 ±67 μs-45.3 μs (-5.7%)NO
    p25825 ±51 μs792 ±57 μs-33.5 μs (-4.1%)NO
    p50876 ±59 μs852 ±58 μs-23.7 μs (-2.7%)NO
    p75972 ±88 μs948 ±82 μs-24.1 μs (-2.5%)NO
    p901.09 ±0.1 ms1.08 ±0.1 ms-10.5 μs (-1.0%)NO

    20 test runs in comparison
    CommitTest Runs
    174e7d8
    • 2023-06-06_19:40:46.437159_kiPG
    • 2023-06-06_19:40:46.441888_ZaRR
    • 2023-06-06_19:40:46.442049_taQy
    • 2023-06-06_19:40:46.442059_mQMZ
    • 2023-06-06_19:40:46.442065_XgyR
    • 2023-06-06_19:40:46.442071_tgtS
    • 2023-06-06_19:40:46.442076_kGKy
    • 2023-06-06_19:40:46.442090_qaPF
    • 2023-06-06_19:40:46.442096_JNez
    • 2023-06-06_19:40:46.442102_eszK
    b7d3504
    • 2023-06-07_15:02:08.787186_wzlv
    • 2023-06-07_15:02:08.793483_bHia
    • 2023-06-07_15:02:08.793506_wFHy
    • 2023-06-07_15:02:08.793514_kise
    • 2023-06-07_15:02:08.793522_CrOP
    • 2023-06-07_15:02:08.793530_SkgK
    • 2023-06-07_15:02:08.793538_MkYO
    • 2023-06-07_15:02:08.793544_DOAe
    • 2023-06-07_15:02:08.793552_ohjc
    • 2023-06-07_15:02:08.793559_soqF
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentile174e7d8b7d3504DiffSignificant (?)
    p10193 ±4 ms195 ±3 ms+2.67 ms (+1.4%)NO
    p25198 ±5 ms201 ±2 ms+2.93 ms (+1.5%)NO
    p50206 ±5 ms209 ±2 ms+3.52 ms (+1.7%)NO
    p75214 ±5 ms219 ±2 ms+4.79 ms (+2.2%)NO
    p90222 ±6 ms232 ±7 ms+10.5 ms (+4.7%)NO

    20 test runs in comparison
    CommitTest Runs
    174e7d8
    • 2023-06-06_19:40:46.437159_kiPG
    • 2023-06-06_19:40:46.441888_ZaRR
    • 2023-06-06_19:40:46.442049_taQy
    • 2023-06-06_19:40:46.442059_mQMZ
    • 2023-06-06_19:40:46.442065_XgyR
    • 2023-06-06_19:40:46.442071_tgtS
    • 2023-06-06_19:40:46.442076_kGKy
    • 2023-06-06_19:40:46.442090_qaPF
    • 2023-06-06_19:40:46.442096_JNez
    • 2023-06-06_19:40:46.442102_eszK
    b7d3504
    • 2023-06-07_15:02:08.787186_wzlv
    • 2023-06-07_15:02:08.793483_bHia
    • 2023-06-07_15:02:08.793506_wFHy
    • 2023-06-07_15:02:08.793514_kise
    • 2023-06-07_15:02:08.793522_CrOP
    • 2023-06-07_15:02:08.793530_SkgK
    • 2023-06-07_15:02:08.793538_MkYO
    • 2023-06-07_15:02:08.793544_DOAe
    • 2023-06-07_15:02:08.793552_ohjc
    • 2023-06-07_15:02:08.793559_soqF
    redfin-30
    Percentile174e7d8b7d3504DiffSignificant (?)
    p10231 ±3 ms255 ±6 ms+24.2 ms (+10.5%)MAYBE
    p25236 ±3 ms261 ±5 ms+24.6 ms (+10.4%)YES
    p50243 ±3 ms269 ±6 ms+25.7 ms (+10.6%)MAYBE
    p75251 ±4 ms279 ±7 ms+27.6 ms (+11.0%)MAYBE
    p90262 ±4 ms291 ±9.7 ms+29.0 ms (+11.1%)MAYBE

    20 test runs in comparison
    CommitTest Runs
    174e7d8
    • 2023-06-06_19:40:46.437159_kiPG
    • 2023-06-06_19:40:46.441888_ZaRR
    • 2023-06-06_19:40:46.442049_taQy
    • 2023-06-06_19:40:46.442059_mQMZ
    • 2023-06-06_19:40:46.442065_XgyR
    • 2023-06-06_19:40:46.442071_tgtS
    • 2023-06-06_19:40:46.442076_kGKy
    • 2023-06-06_19:40:46.442090_qaPF
    • 2023-06-06_19:40:46.442096_JNez
    • 2023-06-06_19:40:46.442102_eszK
    b7d3504
    • 2023-06-07_15:02:08.787186_wzlv
    • 2023-06-07_15:02:08.793483_bHia
    • 2023-06-07_15:02:08.793506_wFHy
    • 2023-06-07_15:02:08.793514_kise
    • 2023-06-07_15:02:08.793522_CrOP
    • 2023-06-07_15:02:08.793530_SkgK
    • 2023-06-07_15:02:08.793538_MkYO
    • 2023-06-07_15:02:08.793544_DOAe
    • 2023-06-07_15:02:08.793552_ohjc
    • 2023-06-07_15:02:08.793559_soqF

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

Copy link
Contributor

@visumickey visumickey left a comment

Choose a reason for hiding this comment

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

Thank you for making those changes.

@mrober mrober merged commit dd04244 into master Jun 7, 2023
davidmotson pushed a commit that referenced this pull request Jun 20, 2023
* Add different defaults for when RC fetch failed

* Add changelog entry

* Format

* Add throttled case to isLastFetchFailed check

* Add rc fetch failed defaults for cpu and memory capturing frequency
@firebase firebase locked and limited conversation to collaborators Jul 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants