Skip to content

Ensure that on-demand fatal events are never processed on the main thread #6113

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 1 commit into from
Jul 18, 2024

Conversation

mrober
Copy link
Contributor

@mrober mrober commented Jul 18, 2024

Ensure that on-demand fatal events are never processed on the main thread.

Also decreased the timeout on main to under 3 seconds, to give more time to other uncaught exception handlers in the chain.

Also fixed some formatting.

Tested cases:
Uncaught exception thrown from the main thread
Uncaught exception thrown from a background thread
ODF triggered on the main thread
ODF triggered on a background thread

Copy link
Contributor

github-actions bot commented Jul 18, 2024

Release note changes

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

Release Notes
firebase-crashlytics
### {{crashlytics}} version 19.0.4 {: #crashlytics_v19-0-4}

* {{feature}} Added the `isCrashlyticsCollectionEnabled` API to check if Crashlytics collection is enabled.
  ([Github #5919](//github.com/firebase/firebase-android-sdk/issues/5919))

* {{fixed}} Ensure that on-demand fatal events are never processed on the main thread.

#### {{crashlytics}} Kotlin extensions version 19.0.4 {: #crashlytics-ktx_v19-0-4}

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

@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

  • firebase-crashlytics

    Overall coverage changed from ? (d583311) to 11.33% (9e0073f) by ?.

    129 individual files with coverage change

    FilenameBase (d583311)Merge (9e0073f)Diff
    AnalyticsDeferredProxy.java?0.00%?
    AnalyticsEventLogger.java?0.00%?
    AnalyticsEventReceiver.java?0.00%?
    AppData.java?50.00%?
    AutoCrashlyticsReportEncoder.java?57.80%?
    AutoRolloutAssignmentEncoder.java?0.00%?
    AutoValue_CrashlyticsReport.java?0.00%?
    AutoValue_CrashlyticsReportWithSessionId.java?0.00%?
    AutoValue_CrashlyticsReport_ApplicationExitInfo.java?58.42%?
    AutoValue_CrashlyticsReport_ApplicationExitInfo_BuildIdMappingForArch.java?0.00%?
    AutoValue_CrashlyticsReport_CustomAttribute.java?0.00%?
    AutoValue_CrashlyticsReport_FilesPayload.java?0.00%?
    AutoValue_CrashlyticsReport_FilesPayload_File.java?0.00%?
    AutoValue_CrashlyticsReport_Session.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Application.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Application_Organization.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Device.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Application.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Application_Execution.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Application_Execution_BinaryImage.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Application_Execution_Exception.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Application_Execution_Signal.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Application_Execution_Thread.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Application_Execution_Thread_Frame.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Application_ProcessDetails.java?40.35%?
    AutoValue_CrashlyticsReport_Session_Event_Device.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_Log.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_RolloutAssignment.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_RolloutAssignment_RolloutVariant.java?0.00%?
    AutoValue_CrashlyticsReport_Session_Event_RolloutsState.java?0.00%?
    AutoValue_CrashlyticsReport_Session_OperatingSystem.java?0.00%?
    AutoValue_CrashlyticsReport_Session_User.java?0.00%?
    AutoValue_InstallIdProvider_InstallIds.java?0.00%?
    AutoValue_RolloutAssignment.java?0.00%?
    AutoValue_StaticSessionData.java?0.00%?
    AutoValue_StaticSessionData_AppData.java?0.00%?
    AutoValue_StaticSessionData_DeviceData.java?0.00%?
    AutoValue_StaticSessionData_OsData.java?0.00%?
    BackgroundPriorityRunnable.java?0.00%?
    BatteryState.java?0.00%?
    BlockingAnalyticsEventLogger.java?0.00%?
    BreadcrumbAnalyticsEventReceiver.java?0.00%?
    BreadcrumbHandler.java?0.00%?
    BreadcrumbSource.java?0.00%?
    BuildIdInfo.java?62.50%?
    BytesBackedNativeSessionFile.java?0.00%?
    CachedSettingsIo.java?0.00%?
    CLSUUID.java?0.00%?
    CommonUtils.java?1.09%?
    CrashlyticsAnalyticsListener.java?0.00%?
    CrashlyticsAppQualitySessionsStore.java?0.00%?
    CrashlyticsAppQualitySessionsSubscriber.java?0.00%?
    CrashlyticsBackgroundWorker.java?56.25%?
    CrashlyticsController.java?16.10%?
    CrashlyticsCore.java?0.61%?
    CrashlyticsFileMarker.java?30.77%?
    CrashlyticsLifecycleEvents.java?0.00%?
    CrashlyticsNativeComponent.java?0.00%?
    CrashlyticsNativeComponentDeferredProxy.java?25.00%?
    CrashlyticsOriginAnalyticsEventLogger.java?0.00%?
    CrashlyticsRegistrar.java?0.00%?
    CrashlyticsRemoteConfigListener.kt?0.00%?
    CrashlyticsReport.java?4.65%?
    CrashlyticsReportDataCapture.java?3.33%?
    CrashlyticsReportJsonTransform.java?1.01%?
    CrashlyticsReportPersistence.java?3.06%?
    CrashlyticsReportWithSessionId.java?0.00%?
    CrashlyticsUncaughtExceptionHandler.java?0.00%?
    CurrentTimeProvider.java?0.00%?
    CustomKeysAndValues.java?0.00%?
    DataCollectionArbiter.java?83.33%?
    DataTransportCrashlyticsReportSender.java?48.28%?
    DefaultSettingsJsonTransform.java?0.00%?
    DefaultSettingsSpiCall.java?0.00%?
    DeliveryMechanism.java?0.00%?
    DevelopmentPlatformProvider.java?0.00%?
    DisabledBreadcrumbSource.java?0.00%?
    ExecutorUtils.java?0.00%?
    FileBackedNativeSessionFile.java?0.00%?
    FileLogStore.java?0.00%?
    FileStore.java?29.85%?
    FirebaseCrashlytics.java?0.00%?
    FirebaseCrashlytics.kt?0.00%?
    FirebaseInstallationId.kt?0.00%?
    HttpGetRequest.java?0.00%?
    HttpRequestFactory.java?0.00%?
    HttpResponse.java?0.00%?
    IdManager.java?2.74%?
    InstallerPackageNameProvider.java?0.00%?
    InstallIdProvider.java?0.00%?
    KeysMap.java?24.32%?
    KeyValueBuilder.kt?0.00%?
    LogFileManager.java?60.71%?
    Logger.java?40.54%?
    MetaDataStore.java?17.83%?
    MiddleOutFallbackStrategy.java?0.00%?
    MiddleOutStrategy.java?0.00%?
    NativeSessionFile.java?0.00%?
    NativeSessionFileGzipper.java?0.00%?
    NativeSessionFileProvider.java?0.00%?
    OnDemandCounter.java?0.00%?
    ProcessDetailsProvider.kt?70.59%?
    QueueFile.java?0.00%?
    QueueFileLogStore.java?7.58%?
    RemoteConfigDeferredProxy.java?0.00%?
    RemoveRepeatsStrategy.java?0.00%?
    ReportQueue.java?0.00%?
    ResponseParser.java?0.00%?
    RolloutAssignment.java?0.00%?
    RolloutAssignmentList.java?44.44%?
    SessionReportingCoordinator.java?42.11%?
    Settings.java?95.00%?
    SettingsCacheBehavior.java?0.00%?
    SettingsController.java?0.00%?
    SettingsJsonConstants.java?0.00%?
    SettingsJsonParser.java?0.00%?
    SettingsJsonTransform.java?0.00%?
    SettingsProvider.java?0.00%?
    SettingsRequest.java?0.00%?
    SettingsSpiCall.java?0.00%?
    SettingsV3JsonTransform.java?0.00%?
    StackTraceTrimmingStrategy.java?0.00%?
    StaticSessionData.java?0.00%?
    SystemCurrentTimeProvider.java?0.00%?
    TrimmedThrowableData.java?0.00%?
    UnavailableAnalyticsEventLogger.java?0.00%?
    UserMetadata.java?22.12%?
    Utils.java?0.00%?

Test Logs

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

Copy link
Contributor

Unit Test Results

  8 files   -   42    8 suites   - 42   24s ⏱️ -59s
22 tests  -   54  22 ✔️  -   54  0 💤 ±0  0 ±0 
44 runs   - 124  44 ✔️  - 124  0 💤 ±0  0 ±0 

Results for commit deee48a. ± Comparison against base commit d583311.

This pull request removes 76 and adds 22 tests. Note that renamed tests count towards both.
com.google.firebase.CoroutinesPlayServicesTests ‑ Task#await() resolves to the same result as Task#getResult()
com.google.firebase.CoroutinesPlayServicesTests ‑ Task#await() throws an Exception for failing Tasks
com.google.firebase.DataCollectionPostNDefaultEnabledTest ‑ isDataCollectionDefaultEnabled_shouldDefaultToTrue
com.google.firebase.DataCollectionPostNDefaultEnabledTest ‑ isDataCollectionDefaultEnabled_whenPrefsFalse_shouldReturnFalse
com.google.firebase.DataCollectionPostNDefaultEnabledTest ‑ isDataCollectionDefaultEnabled_whenPrefsTrue_shouldReturnTrue
com.google.firebase.DataCollectionPostNDefaultEnabledTest ‑ setDataCollectionDefaultEnabledFalse_shouldUpdateSharedPrefs
com.google.firebase.DataCollectionPostNDefaultEnabledTest ‑ setDataCollectionDefaultEnabled_shouldNotAffectOtherFirebaseAppInstances
com.google.firebase.DataCollectionPreNDefaultEnabledTest ‑ isDataCollectionDefaultEnabled_shouldDefaultToTrue
com.google.firebase.DataCollectionPreNDefaultEnabledTest ‑ isDataCollectionDefaultEnabled_whenPrefsFalse_shouldReturnFalse
com.google.firebase.DataCollectionPreNDefaultEnabledTest ‑ isDataCollectionDefaultEnabled_whenPrefsTrue_shouldReturnTrue
…
com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_disabledAnrs_doesNotPersistsAppExitInfo
com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_enabledAnrs_doesNotPersistsAppExitInfoIfItDoesntExist
com.google.firebase.crashlytics.internal.common.CrashlyticsControllerRobolectricTest ‑ testDoCloseSession_enabledAnrs_persistsAppExitInfoIfItExists
com.google.firebase.crashlytics.internal.common.DataCollectionArbiterRobolectricTest ‑ testDefaultDataCollection_usedWhenNoOverrideOrManifestSetting
com.google.firebase.crashlytics.internal.common.DataCollectionArbiterRobolectricTest ‑ testManifestMetadata_respectedWhenNoOverride
com.google.firebase.crashlytics.internal.common.DataCollectionArbiterRobolectricTest ‑ testSetCrashlyticsDataCollectionEnabled_overridesOtherSettings
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_notPersistIfAnrBeforeSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_notPersistIfAppExitInfoNotAnrButWithinSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_persistIfAnrWithinSession
com.google.firebase.crashlytics.internal.common.SessionReportingCoordinatorRobolectricTest ‑ testAppExitInfoEvent_persistIfAnrWithinSession_multipleAppExitInfo
…

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • firebase-crashlytics

    TypeBase (d583311)Merge (9e0073f)Diff
    aar408 kB408 kB+42 B (+0.0%)
    apk (release)5.84 MB5.84 MB+52 B (+0.0%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/jHnVG9X0Iw.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

  • fire-cls

    DeviceStatisticsDistributions
    oriole-32
    Percentiled5833119e0073fDiffSignificant (?)
    p1011.8 ±2 ms12.6 ±3 ms+818 μs (+6.9%)NO
    p2512.4 ±3 ms13.4 ±4 ms+925 μs (+7.4%)NO
    p5013.4 ±3 ms14.5 ±4 ms+1.11 ms (+8.3%)NO
    p7514.7 ±3 ms15.9 ±4 ms+1.19 ms (+8.1%)NO
    p9016.4 ±3 ms17.6 ±4 ms+1.14 ms (+6.9%)NO

    20 test runs in comparison
    CommitTest Runs
    d583311
    • 2024-07-17_21:29:14.897099_HArT
    • 2024-07-17_21:29:14.897142_QXRq
    • 2024-07-17_21:29:14.897152_YdwI
    • 2024-07-17_21:29:14.897160_pnid
    • 2024-07-17_21:29:14.897168_utlb
    • 2024-07-17_21:29:14.897183_ywGx
    • 2024-07-17_21:29:14.897192_sCtp
    • 2024-07-17_21:29:14.897199_bWBc
    • 2024-07-17_21:29:14.897207_GRwT
    • 2024-07-17_21:29:14.897213_rfhT
    9e0073f
    • 2024-07-18_17:23:41.551310_jkQx
    • 2024-07-18_17:23:41.551360_HSyK
    • 2024-07-18_17:23:41.551371_bGSg
    • 2024-07-18_17:23:41.551386_plMX
    • 2024-07-18_17:23:41.551393_SZFR
    • 2024-07-18_17:23:41.551401_Vjze
    • 2024-07-18_17:23:41.551407_HHqh
    • 2024-07-18_17:23:41.551414_xOTR
    • 2024-07-18_17:23:41.551421_ymKi
    • 2024-07-18_17:23:41.551427_UGht
    redfin-30
    Percentiled5833119e0073fDiffSignificant (?)
    p1013.9 ±4 ms14.2 ±3 ms+213 μs (+1.5%)NO
    p2514.4 ±5 ms14.9 ±3 ms+503 μs (+3.5%)NO
    p5015.2 ±5 ms16.1 ±4 ms+888 μs (+5.8%)NO
    p7516.4 ±6 ms17.5 ±5 ms+1.14 ms (+7.0%)NO
    p9018.1 ±7 ms20.0 ±6 ms+1.95 ms (+10.8%)NO

    20 test runs in comparison
    CommitTest Runs
    d583311
    • 2024-07-17_21:29:14.897099_HArT
    • 2024-07-17_21:29:14.897142_QXRq
    • 2024-07-17_21:29:14.897152_YdwI
    • 2024-07-17_21:29:14.897160_pnid
    • 2024-07-17_21:29:14.897168_utlb
    • 2024-07-17_21:29:14.897183_ywGx
    • 2024-07-17_21:29:14.897192_sCtp
    • 2024-07-17_21:29:14.897199_bWBc
    • 2024-07-17_21:29:14.897207_GRwT
    • 2024-07-17_21:29:14.897213_rfhT
    9e0073f
    • 2024-07-18_17:23:41.551310_jkQx
    • 2024-07-18_17:23:41.551360_HSyK
    • 2024-07-18_17:23:41.551371_bGSg
    • 2024-07-18_17:23:41.551386_plMX
    • 2024-07-18_17:23:41.551393_SZFR
    • 2024-07-18_17:23:41.551401_Vjze
    • 2024-07-18_17:23:41.551407_HHqh
    • 2024-07-18_17:23:41.551414_xOTR
    • 2024-07-18_17:23:41.551421_ymKi
    • 2024-07-18_17:23:41.551427_UGht
  • fire-rc

    DeviceStatisticsDistributions
    oriole-32
    Percentiled5833119e0073fDiffSignificant (?)
    p10239 ±390 μs304 ±461 μs+64.7 μs (+27.1%)NO
    p25249 ±404 μs314 ±473 μs+65.4 μs (+26.3%)NO
    p50267 ±428 μs343 ±507 μs+76.5 μs (+28.7%)NO
    p75299 ±476 μs392 ±558 μs+93.2 μs (+31.2%)NO
    p90394 ±575 μs479 ±647 μs+85.0 μs (+21.6%)NO

    20 test runs in comparison
    CommitTest Runs
    d583311
    • 2024-07-17_21:29:14.897099_HArT
    • 2024-07-17_21:29:14.897142_QXRq
    • 2024-07-17_21:29:14.897152_YdwI
    • 2024-07-17_21:29:14.897160_pnid
    • 2024-07-17_21:29:14.897168_utlb
    • 2024-07-17_21:29:14.897183_ywGx
    • 2024-07-17_21:29:14.897192_sCtp
    • 2024-07-17_21:29:14.897199_bWBc
    • 2024-07-17_21:29:14.897207_GRwT
    • 2024-07-17_21:29:14.897213_rfhT
    9e0073f
    • 2024-07-18_17:23:41.551310_jkQx
    • 2024-07-18_17:23:41.551360_HSyK
    • 2024-07-18_17:23:41.551371_bGSg
    • 2024-07-18_17:23:41.551386_plMX
    • 2024-07-18_17:23:41.551393_SZFR
    • 2024-07-18_17:23:41.551401_Vjze
    • 2024-07-18_17:23:41.551407_HHqh
    • 2024-07-18_17:23:41.551414_xOTR
    • 2024-07-18_17:23:41.551421_ymKi
    • 2024-07-18_17:23:41.551427_UGht
    redfin-30
    Percentiled5833119e0073fDiffSignificant (?)
    p10560 ±667 μs298 ±328 μs-262 μs (-46.7%)NO
    p25640 ±770 μs355 ±432 μs-285 μs (-44.6%)NO
    p50774 ±906 μs424 ±562 μs-349 μs (-45.2%)NO
    p75933 ±1064 μs552 ±801 μs-381 μs (-40.9%)NO
    p901.18 ±1 ms756 ±1151 μs-423 μs (-35.9%)NO

    20 test runs in comparison
    CommitTest Runs
    d583311
    • 2024-07-17_21:29:14.897099_HArT
    • 2024-07-17_21:29:14.897142_QXRq
    • 2024-07-17_21:29:14.897152_YdwI
    • 2024-07-17_21:29:14.897160_pnid
    • 2024-07-17_21:29:14.897168_utlb
    • 2024-07-17_21:29:14.897183_ywGx
    • 2024-07-17_21:29:14.897192_sCtp
    • 2024-07-17_21:29:14.897199_bWBc
    • 2024-07-17_21:29:14.897207_GRwT
    • 2024-07-17_21:29:14.897213_rfhT
    9e0073f
    • 2024-07-18_17:23:41.551310_jkQx
    • 2024-07-18_17:23:41.551360_HSyK
    • 2024-07-18_17:23:41.551371_bGSg
    • 2024-07-18_17:23:41.551386_plMX
    • 2024-07-18_17:23:41.551393_SZFR
    • 2024-07-18_17:23:41.551401_Vjze
    • 2024-07-18_17:23:41.551407_HHqh
    • 2024-07-18_17:23:41.551414_xOTR
    • 2024-07-18_17:23:41.551421_ymKi
    • 2024-07-18_17:23:41.551427_UGht
  • fire-sessions

    DeviceStatisticsDistributions
    oriole-32
    Percentiled5833119e0073fDiffSignificant (?)
    p102.63 ±0.6 ms3.03 ±0.8 ms+405 μs (+15.4%)NO
    p252.76 ±0.6 ms3.24 ±0.8 ms+482 μs (+17.5%)NO
    p503.09 ±0.7 ms3.61 ±1 ms+517 μs (+16.7%)NO
    p753.62 ±0.9 ms4.12 ±1 ms+497 μs (+13.7%)NO
    p904.33 ±1 ms4.92 ±1 ms+586 μs (+13.5%)NO

    20 test runs in comparison
    CommitTest Runs
    d583311
    • 2024-07-17_21:29:14.897099_HArT
    • 2024-07-17_21:29:14.897142_QXRq
    • 2024-07-17_21:29:14.897152_YdwI
    • 2024-07-17_21:29:14.897160_pnid
    • 2024-07-17_21:29:14.897168_utlb
    • 2024-07-17_21:29:14.897183_ywGx
    • 2024-07-17_21:29:14.897192_sCtp
    • 2024-07-17_21:29:14.897199_bWBc
    • 2024-07-17_21:29:14.897207_GRwT
    • 2024-07-17_21:29:14.897213_rfhT
    9e0073f
    • 2024-07-18_17:23:41.551310_jkQx
    • 2024-07-18_17:23:41.551360_HSyK
    • 2024-07-18_17:23:41.551371_bGSg
    • 2024-07-18_17:23:41.551386_plMX
    • 2024-07-18_17:23:41.551393_SZFR
    • 2024-07-18_17:23:41.551401_Vjze
    • 2024-07-18_17:23:41.551407_HHqh
    • 2024-07-18_17:23:41.551414_xOTR
    • 2024-07-18_17:23:41.551421_ymKi
    • 2024-07-18_17:23:41.551427_UGht
    redfin-30
    Percentiled5833119e0073fDiffSignificant (?)
    p106.30 ±1 ms6.58 ±2 ms+280 μs (+4.4%)NO
    p256.75 ±1 ms7.10 ±2 ms+342 μs (+5.1%)NO
    p507.36 ±2 ms7.97 ±3 ms+612 μs (+8.3%)NO
    p758.35 ±3 ms8.98 ±4 ms+627 μs (+7.5%)NO
    p909.62 ±4 ms9.95 ±5 ms+338 μs (+3.5%)NO

    20 test runs in comparison
    CommitTest Runs
    d583311
    • 2024-07-17_21:29:14.897099_HArT
    • 2024-07-17_21:29:14.897142_QXRq
    • 2024-07-17_21:29:14.897152_YdwI
    • 2024-07-17_21:29:14.897160_pnid
    • 2024-07-17_21:29:14.897168_utlb
    • 2024-07-17_21:29:14.897183_ywGx
    • 2024-07-17_21:29:14.897192_sCtp
    • 2024-07-17_21:29:14.897199_bWBc
    • 2024-07-17_21:29:14.897207_GRwT
    • 2024-07-17_21:29:14.897213_rfhT
    9e0073f
    • 2024-07-18_17:23:41.551310_jkQx
    • 2024-07-18_17:23:41.551360_HSyK
    • 2024-07-18_17:23:41.551371_bGSg
    • 2024-07-18_17:23:41.551386_plMX
    • 2024-07-18_17:23:41.551393_SZFR
    • 2024-07-18_17:23:41.551401_Vjze
    • 2024-07-18_17:23:41.551407_HHqh
    • 2024-07-18_17:23:41.551414_xOTR
    • 2024-07-18_17:23:41.551421_ymKi
    • 2024-07-18_17:23:41.551427_UGht
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentiled5833119e0073fDiffSignificant (?)
    p10199 ±3 ms208 ±6 ms+8.79 ms (+4.4%)NO
    p25205 ±3 ms214 ±7 ms+9.35 ms (+4.6%)NO
    p50212 ±3 ms221 ±7 ms+9.10 ms (+4.3%)NO
    p75220 ±3 ms228 ±7 ms+8.91 ms (+4.1%)NO
    p90229 ±4 ms239 ±7 ms+9.56 ms (+4.2%)NO

    20 test runs in comparison
    CommitTest Runs
    d583311
    • 2024-07-17_21:29:14.897099_HArT
    • 2024-07-17_21:29:14.897142_QXRq
    • 2024-07-17_21:29:14.897152_YdwI
    • 2024-07-17_21:29:14.897160_pnid
    • 2024-07-17_21:29:14.897168_utlb
    • 2024-07-17_21:29:14.897183_ywGx
    • 2024-07-17_21:29:14.897192_sCtp
    • 2024-07-17_21:29:14.897199_bWBc
    • 2024-07-17_21:29:14.897207_GRwT
    • 2024-07-17_21:29:14.897213_rfhT
    9e0073f
    • 2024-07-18_17:23:41.551310_jkQx
    • 2024-07-18_17:23:41.551360_HSyK
    • 2024-07-18_17:23:41.551371_bGSg
    • 2024-07-18_17:23:41.551386_plMX
    • 2024-07-18_17:23:41.551393_SZFR
    • 2024-07-18_17:23:41.551401_Vjze
    • 2024-07-18_17:23:41.551407_HHqh
    • 2024-07-18_17:23:41.551414_xOTR
    • 2024-07-18_17:23:41.551421_ymKi
    • 2024-07-18_17:23:41.551427_UGht
    redfin-30
    Percentiled5833119e0073fDiffSignificant (?)
    p10243 ±4 ms270 ±8 ms+26.4 ms (+10.8%)MAYBE
    p25248 ±5 ms276 ±8 ms+27.8 ms (+11.2%)MAYBE
    p50254 ±5 ms284 ±9 ms+29.5 ms (+11.6%)MAYBE
    p75262 ±5 ms294 ±12 ms+31.9 ms (+12.2%)MAYBE
    p90272 ±7 ms308 ±15 ms+36.2 ms (+13.3%)NO

    20 test runs in comparison
    CommitTest Runs
    d583311
    • 2024-07-17_21:29:14.897099_HArT
    • 2024-07-17_21:29:14.897142_QXRq
    • 2024-07-17_21:29:14.897152_YdwI
    • 2024-07-17_21:29:14.897160_pnid
    • 2024-07-17_21:29:14.897168_utlb
    • 2024-07-17_21:29:14.897183_ywGx
    • 2024-07-17_21:29:14.897192_sCtp
    • 2024-07-17_21:29:14.897199_bWBc
    • 2024-07-17_21:29:14.897207_GRwT
    • 2024-07-17_21:29:14.897213_rfhT
    9e0073f
    • 2024-07-18_17:23:41.551310_jkQx
    • 2024-07-18_17:23:41.551360_HSyK
    • 2024-07-18_17:23:41.551371_bGSg
    • 2024-07-18_17:23:41.551386_plMX
    • 2024-07-18_17:23:41.551393_SZFR
    • 2024-07-18_17:23:41.551401_Vjze
    • 2024-07-18_17:23:41.551407_HHqh
    • 2024-07-18_17:23:41.551414_xOTR
    • 2024-07-18_17:23:41.551421_ymKi
    • 2024-07-18_17:23:41.551427_UGht

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

@mrober mrober merged commit 647d88c into main Jul 18, 2024
38 checks passed
@mrober mrober deleted the crashlytics-odfs2 branch July 18, 2024 18:03
@firebase firebase locked and limited conversation to collaborators Aug 18, 2024
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