Skip to content
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

Make Logger a normal abstract class #5474

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Make Logger a normal abstract class #5474

merged 1 commit into from
Oct 26, 2023

Conversation

mrober
Copy link
Contributor

@mrober mrober commented Oct 26, 2023

Discussed in #5469 (comment)

@github-actions
Copy link
Contributor

github-actions bot commented Oct 26, 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.

1 similar comment
@github-actions
Copy link
Contributor

📝 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

@github-actions
Copy link
Contributor

Unit Test Results

  46 files   -    134    46 suites   - 134   1m 43s ⏱️ -36s
  66 tests  - 1 156    66 ✔️  - 1 140  0 💤  - 16  0 ±0 
148 runs   - 2 320  148 ✔️  - 2 288  0 💤  - 32  0 ±0 

Results for commit 934dcb9. ± Comparison against base commit f6b5ecb.

@mrober mrober merged commit 14d7bef into master Oct 26, 2023
27 checks passed
@mrober mrober deleted the common-logger-unsealed branch October 26, 2023 13:42
@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • firebase-common

    TypeBase (f6b5ecb)Merge (2d79654)Diff
    aar107 kB107 kB-23 B (-0.0%)
    apk (release)1.65 MB1.65 MB+56 B (+0.0%)

Test Logs

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

@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
    Percentilef6b5ecb2d79654DiffSignificant (?)
    p10941 ±59 μs930 ±64 μs-11.0 μs (-1.2%)NO
    p251.02 ±0.07 ms1.03 ±0.06 ms+2.39 μs (+0.2%)NO
    p501.16 ±0.09 ms1.15 ±0.07 ms-13.3 μs (-1.1%)NO
    p751.38 ±0.1 ms1.34 ±0.08 ms-32.5 μs (-2.4%)NO
    p901.68 ±0.08 ms1.61 ±0.1 ms-65.0 μs (-3.9%)NO

    20 test runs in comparison
    CommitTest Runs
    f6b5ecb
    • 2023-10-25_19:52:19.534770_OROG
    • 2023-10-25_19:52:19.536869_Tlmb
    • 2023-10-25_19:52:19.536887_rMri
    • 2023-10-25_19:52:19.536895_IKvK
    • 2023-10-25_19:52:19.536907_CeRa
    • 2023-10-25_19:52:19.536915_IdTs
    • 2023-10-25_19:52:19.536922_QbLo
    • 2023-10-25_19:52:19.536929_kqUI
    • 2023-10-25_19:52:19.536936_XIgb
    • 2023-10-25_19:52:19.536958_pNGb
    2d79654
    • 2023-10-26_13:46:27.134157_HAve
    • 2023-10-26_13:46:27.145520_ybLS
    • 2023-10-26_13:46:27.145536_TCQo
    • 2023-10-26_13:46:27.145543_wszz
    • 2023-10-26_13:46:27.145548_PMVr
    • 2023-10-26_13:46:27.145554_RNEc
    • 2023-10-26_13:46:27.145559_COYa
    • 2023-10-26_13:46:27.145565_WzCO
    • 2023-10-26_13:46:27.145570_Dwuf
    • 2023-10-26_13:46:27.145576_vlnb
    redfin-30
    Percentilef6b5ecb2d79654DiffSignificant (?)
    p10861 ±14 μs878 ±69 μs+17.5 μs (+2.0%)NO
    p25901 ±18 μs919 ±73 μs+18.4 μs (+2.0%)NO
    p50976 ±36 μs991 ±68 μs+14.8 μs (+1.5%)NO
    p751.24 ±0.07 ms1.28 ±0.08 ms+38.2 μs (+3.1%)NO
    p901.83 ±0.2 ms1.77 ±0.3 ms-58.0 μs (-3.2%)NO

    20 test runs in comparison
    CommitTest Runs
    f6b5ecb
    • 2023-10-25_19:52:19.534770_OROG
    • 2023-10-25_19:52:19.536869_Tlmb
    • 2023-10-25_19:52:19.536887_rMri
    • 2023-10-25_19:52:19.536895_IKvK
    • 2023-10-25_19:52:19.536907_CeRa
    • 2023-10-25_19:52:19.536915_IdTs
    • 2023-10-25_19:52:19.536922_QbLo
    • 2023-10-25_19:52:19.536929_kqUI
    • 2023-10-25_19:52:19.536936_XIgb
    • 2023-10-25_19:52:19.536958_pNGb
    2d79654
    • 2023-10-26_13:46:27.134157_HAve
    • 2023-10-26_13:46:27.145520_ybLS
    • 2023-10-26_13:46:27.145536_TCQo
    • 2023-10-26_13:46:27.145543_wszz
    • 2023-10-26_13:46:27.145548_PMVr
    • 2023-10-26_13:46:27.145554_RNEc
    • 2023-10-26_13:46:27.145559_COYa
    • 2023-10-26_13:46:27.145565_WzCO
    • 2023-10-26_13:46:27.145570_Dwuf
    • 2023-10-26_13:46:27.145576_vlnb
  • Firebase

    DeviceStatisticsDistributions
    oriole-32
    Percentilef6b5ecb2d79654DiffSignificant (?)
    p106.06 ±0.1 ms6.07 ±0.2 ms+9.40 μs (+0.2%)NO
    p256.41 ±0.2 ms6.34 ±0.2 ms-69.6 μs (-1.1%)NO
    p506.91 ±0.2 ms6.83 ±0.2 ms-74.6 μs (-1.1%)NO
    p757.51 ±0.2 ms7.48 ±0.3 ms-27.2 μs (-0.4%)NO
    p908.42 ±0.4 ms8.24 ±0.4 ms-180 μs (-2.1%)NO

    20 test runs in comparison
    CommitTest Runs
    f6b5ecb
    • 2023-10-25_19:52:19.534770_OROG
    • 2023-10-25_19:52:19.536869_Tlmb
    • 2023-10-25_19:52:19.536887_rMri
    • 2023-10-25_19:52:19.536895_IKvK
    • 2023-10-25_19:52:19.536907_CeRa
    • 2023-10-25_19:52:19.536915_IdTs
    • 2023-10-25_19:52:19.536922_QbLo
    • 2023-10-25_19:52:19.536929_kqUI
    • 2023-10-25_19:52:19.536936_XIgb
    • 2023-10-25_19:52:19.536958_pNGb
    2d79654
    • 2023-10-26_13:46:27.134157_HAve
    • 2023-10-26_13:46:27.145520_ybLS
    • 2023-10-26_13:46:27.145536_TCQo
    • 2023-10-26_13:46:27.145543_wszz
    • 2023-10-26_13:46:27.145548_PMVr
    • 2023-10-26_13:46:27.145554_RNEc
    • 2023-10-26_13:46:27.145559_COYa
    • 2023-10-26_13:46:27.145565_WzCO
    • 2023-10-26_13:46:27.145570_Dwuf
    • 2023-10-26_13:46:27.145576_vlnb
    redfin-30
    Percentilef6b5ecb2d79654DiffSignificant (?)
    p108.33 ±0.2 ms8.49 ±0.7 ms+155 μs (+1.9%)NO
    p258.60 ±0.2 ms8.83 ±0.9 ms+238 μs (+2.8%)NO
    p508.97 ±0.3 ms9.28 ±1 ms+308 μs (+3.4%)NO
    p759.88 ±0.3 ms10.1 ±0.9 ms+209 μs (+2.1%)NO
    p9011.3 ±0.6 ms11.6 ±1 ms+245 μs (+2.2%)NO

    20 test runs in comparison
    CommitTest Runs
    f6b5ecb
    • 2023-10-25_19:52:19.534770_OROG
    • 2023-10-25_19:52:19.536869_Tlmb
    • 2023-10-25_19:52:19.536887_rMri
    • 2023-10-25_19:52:19.536895_IKvK
    • 2023-10-25_19:52:19.536907_CeRa
    • 2023-10-25_19:52:19.536915_IdTs
    • 2023-10-25_19:52:19.536922_QbLo
    • 2023-10-25_19:52:19.536929_kqUI
    • 2023-10-25_19:52:19.536936_XIgb
    • 2023-10-25_19:52:19.536958_pNGb
    2d79654
    • 2023-10-26_13:46:27.134157_HAve
    • 2023-10-26_13:46:27.145520_ybLS
    • 2023-10-26_13:46:27.145536_TCQo
    • 2023-10-26_13:46:27.145543_wszz
    • 2023-10-26_13:46:27.145548_PMVr
    • 2023-10-26_13:46:27.145554_RNEc
    • 2023-10-26_13:46:27.145559_COYa
    • 2023-10-26_13:46:27.145565_WzCO
    • 2023-10-26_13:46:27.145570_Dwuf
    • 2023-10-26_13:46:27.145576_vlnb
  • Runtime

    DeviceStatisticsDistributions
    oriole-32
    Percentilef6b5ecb2d79654DiffSignificant (?)
    p104.98 ±0.08 ms4.96 ±0.1 ms-16.4 μs (-0.3%)NO
    p255.22 ±0.1 ms5.18 ±0.2 ms-36.0 μs (-0.7%)NO
    p505.61 ±0.1 ms5.54 ±0.1 ms-64.9 μs (-1.2%)NO
    p756.14 ±0.2 ms6.08 ±0.2 ms-66.4 μs (-1.1%)NO
    p906.94 ±0.5 ms6.76 ±0.3 ms-182 μs (-2.6%)NO

    20 test runs in comparison
    CommitTest Runs
    f6b5ecb
    • 2023-10-25_19:52:19.534770_OROG
    • 2023-10-25_19:52:19.536869_Tlmb
    • 2023-10-25_19:52:19.536887_rMri
    • 2023-10-25_19:52:19.536895_IKvK
    • 2023-10-25_19:52:19.536907_CeRa
    • 2023-10-25_19:52:19.536915_IdTs
    • 2023-10-25_19:52:19.536922_QbLo
    • 2023-10-25_19:52:19.536929_kqUI
    • 2023-10-25_19:52:19.536936_XIgb
    • 2023-10-25_19:52:19.536958_pNGb
    2d79654
    • 2023-10-26_13:46:27.134157_HAve
    • 2023-10-26_13:46:27.145520_ybLS
    • 2023-10-26_13:46:27.145536_TCQo
    • 2023-10-26_13:46:27.145543_wszz
    • 2023-10-26_13:46:27.145548_PMVr
    • 2023-10-26_13:46:27.145554_RNEc
    • 2023-10-26_13:46:27.145559_COYa
    • 2023-10-26_13:46:27.145565_WzCO
    • 2023-10-26_13:46:27.145570_Dwuf
    • 2023-10-26_13:46:27.145576_vlnb
    redfin-30
    Percentilef6b5ecb2d79654DiffSignificant (?)
    p107.30 ±0.2 ms7.42 ±0.6 ms+128 μs (+1.7%)NO
    p257.52 ±0.2 ms7.74 ±0.8 ms+220 μs (+2.9%)NO
    p507.80 ±0.2 ms8.10 ±0.9 ms+297 μs (+3.8%)NO
    p758.36 ±0.3 ms8.66 ±0.9 ms+299 μs (+3.6%)NO
    p909.32 ±0.6 ms9.76 ±1 ms+441 μs (+4.7%)NO

    20 test runs in comparison
    CommitTest Runs
    f6b5ecb
    • 2023-10-25_19:52:19.534770_OROG
    • 2023-10-25_19:52:19.536869_Tlmb
    • 2023-10-25_19:52:19.536887_rMri
    • 2023-10-25_19:52:19.536895_IKvK
    • 2023-10-25_19:52:19.536907_CeRa
    • 2023-10-25_19:52:19.536915_IdTs
    • 2023-10-25_19:52:19.536922_QbLo
    • 2023-10-25_19:52:19.536929_kqUI
    • 2023-10-25_19:52:19.536936_XIgb
    • 2023-10-25_19:52:19.536958_pNGb
    2d79654
    • 2023-10-26_13:46:27.134157_HAve
    • 2023-10-26_13:46:27.145520_ybLS
    • 2023-10-26_13:46:27.145536_TCQo
    • 2023-10-26_13:46:27.145543_wszz
    • 2023-10-26_13:46:27.145548_PMVr
    • 2023-10-26_13:46:27.145554_RNEc
    • 2023-10-26_13:46:27.145559_COYa
    • 2023-10-26_13:46:27.145565_WzCO
    • 2023-10-26_13:46:27.145570_Dwuf
    • 2023-10-26_13:46:27.145576_vlnb
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentilef6b5ecb2d79654DiffSignificant (?)
    p10201 ±3 ms205 ±4 ms+4.66 ms (+2.3%)NO
    p25207 ±3 ms211 ±3 ms+3.99 ms (+1.9%)NO
    p50214 ±4 ms218 ±4 ms+3.14 ms (+1.5%)NO
    p75223 ±4 ms226 ±5 ms+2.84 ms (+1.3%)NO
    p90231 ±4 ms237 ±7 ms+5.73 ms (+2.5%)NO

    20 test runs in comparison
    CommitTest Runs
    f6b5ecb
    • 2023-10-25_19:52:19.534770_OROG
    • 2023-10-25_19:52:19.536869_Tlmb
    • 2023-10-25_19:52:19.536887_rMri
    • 2023-10-25_19:52:19.536895_IKvK
    • 2023-10-25_19:52:19.536907_CeRa
    • 2023-10-25_19:52:19.536915_IdTs
    • 2023-10-25_19:52:19.536922_QbLo
    • 2023-10-25_19:52:19.536929_kqUI
    • 2023-10-25_19:52:19.536936_XIgb
    • 2023-10-25_19:52:19.536958_pNGb
    2d79654
    • 2023-10-26_13:46:27.134157_HAve
    • 2023-10-26_13:46:27.145520_ybLS
    • 2023-10-26_13:46:27.145536_TCQo
    • 2023-10-26_13:46:27.145543_wszz
    • 2023-10-26_13:46:27.145548_PMVr
    • 2023-10-26_13:46:27.145554_RNEc
    • 2023-10-26_13:46:27.145559_COYa
    • 2023-10-26_13:46:27.145565_WzCO
    • 2023-10-26_13:46:27.145570_Dwuf
    • 2023-10-26_13:46:27.145576_vlnb
    redfin-30
    Percentilef6b5ecb2d79654DiffSignificant (?)
    p10243 ±4 ms272 ±16 ms+28.9 ms (+11.9%)NO
    p25249 ±3 ms280 ±21 ms+31.7 ms (+12.7%)NO
    p50256 ±3 ms292 ±31 ms+36.0 ms (+14.1%)NO
    p75265 ±4 ms303 ±35 ms+38.8 ms (+14.7%)NO
    p90274 ±5 ms318 ±37 ms+43.9 ms (+16.0%)NO

    20 test runs in comparison
    CommitTest Runs
    f6b5ecb
    • 2023-10-25_19:52:19.534770_OROG
    • 2023-10-25_19:52:19.536869_Tlmb
    • 2023-10-25_19:52:19.536887_rMri
    • 2023-10-25_19:52:19.536895_IKvK
    • 2023-10-25_19:52:19.536907_CeRa
    • 2023-10-25_19:52:19.536915_IdTs
    • 2023-10-25_19:52:19.536922_QbLo
    • 2023-10-25_19:52:19.536929_kqUI
    • 2023-10-25_19:52:19.536936_XIgb
    • 2023-10-25_19:52:19.536958_pNGb
    2d79654
    • 2023-10-26_13:46:27.134157_HAve
    • 2023-10-26_13:46:27.145520_ybLS
    • 2023-10-26_13:46:27.145536_TCQo
    • 2023-10-26_13:46:27.145543_wszz
    • 2023-10-26_13:46:27.145548_PMVr
    • 2023-10-26_13:46:27.145554_RNEc
    • 2023-10-26_13:46:27.145559_COYa
    • 2023-10-26_13:46:27.145565_WzCO
    • 2023-10-26_13:46:27.145570_Dwuf
    • 2023-10-26_13:46:27.145576_vlnb

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

@firebase firebase locked and limited conversation to collaborators Nov 26, 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