Skip to content

Clarify multi db documentation #5158

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
Jul 14, 2023
Merged

Conversation

tom-andersen
Copy link
Contributor

@tom-andersen tom-andersen commented Jul 12, 2023

This is to address comment in cl/545750836.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 12, 2023

Javadoc Changes:
--- /Users/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/firestore/FirebaseFirestore.html	2023-07-12 15:34:15.000000000 +0000
+++ /Users/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/firestore/FirebaseFirestore.html	2023-07-12 15:27:43.000000000 +0000
@@ -652,14 +652,14 @@
         <h3 class="api-name" id="getInstance()">getInstance</h3>
         <pre class="api-signature no-pretty-print">public&nbsp;static&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html#getInstance()">getInstance</a>()</pre>
         <p>Returns the default <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance for the default <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>. </p>
-        <p>Returns the same instance for all invocations. If no instance exists, initializes a new instance with default settings.</p>
+        <p>Returns the same instance for all invocations. If no instance exists, initializes a new instance.</p>
         <p>The <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance.</p>
       </div>
       <div class="api-item"><a name="getInstance-com.google.firebase.FirebaseApp-"></a><a name="getinstance"></a>
         <h3 class="api-name" id="getInstance(com.google.firebase.FirebaseApp)">getInstance</h3>
         <pre class="api-signature no-pretty-print">public&nbsp;static&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html#getInstance(com.google.firebase.FirebaseApp)">getInstance</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a>&nbsp;app)</pre>
         <p>Returns the default <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance for the provided <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>. </p>
-        <p>For a given <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>, invocation always returns the same instance. If no instance exists, initializes a new instance with default settings.</p>
+        <p>For a given <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>, invocation always returns the same instance. If no instance exists, initializes a new instance.</p>
         <p>The <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance.</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
@@ -676,7 +676,7 @@
               <tr>
                 <td><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a>&nbsp;app</code></td>
                 <td>
-                  <p>- The <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> instance that the returned <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance is associated with.</p>
+                  <p>The <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> instance that the returned <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance is associated with.</p>
                 </td>
               </tr>
             </tbody>
@@ -687,7 +687,7 @@
         <h3 class="api-name" id="getInstance(java.lang.String)">getInstance</h3>
         <pre class="api-signature no-pretty-print">public&nbsp;static&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html#getInstance(java.lang.String)">getInstance</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/lang/String.html">String</a>&nbsp;database)</pre>
         <p>Returns the <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance for the default <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>. </p>
-        <p>Returns the same instance for all invocations given the same database parameter. If no instance exists, initializes a new instance with default settings.</p>
+        <p>Returns the same instance for all invocations given the same database parameter. If no instance exists, initializes a new instance.</p>
         <p>The <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance.</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
@@ -704,7 +704,7 @@
               <tr>
                 <td><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/lang/String.html">String</a>&nbsp;database</code></td>
                 <td>
-                  <p>- The name of database.</p>
+                  <p>The database ID.</p>
                 </td>
               </tr>
             </tbody>
@@ -715,7 +715,7 @@
         <h3 class="api-name" id="getInstance(com.google.firebase.FirebaseApp,java.lang.String)">getInstance</h3>
         <pre class="api-signature no-pretty-print">public&nbsp;static&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html#getInstance(com.google.firebase.FirebaseApp,java.lang.String)">getInstance</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a>&nbsp;app,&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/lang/String.html">String</a>&nbsp;database)</pre>
         <p>Returns the <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance for the provided <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>. </p>
-        <p>Returns the same instance for all invocations given the same <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> and database parameter. If no instance exists, initializes a new instance with default settings.</p>
+        <p>Returns the same instance for all invocations given the same <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> and database parameter. If no instance exists, initializes a new instance.</p>
         <p>The <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance.</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
@@ -732,13 +732,13 @@
               <tr>
                 <td><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a>&nbsp;app</code></td>
                 <td>
-                  <p>- The <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> instance that the returned <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance is associated with.</p>
+                  <p>The <code><a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> instance that the returned <code><a href="/docs/reference/android/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance is associated with.</p>
                 </td>
               </tr>
               <tr>
                 <td><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/lang/String.html">String</a>&nbsp;database</code></td>
                 <td>
-                  <p>- The name of database.</p>
+                  <p>The database ID.</p>
                 </td>
               </tr>
             </tbody>
--- /Users/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/firestore/FirebaseFirestore.html	2023-07-12 15:34:15.000000000 +0000
+++ /Users/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/firestore/FirebaseFirestore.html	2023-07-12 15:27:43.000000000 +0000
@@ -652,14 +652,14 @@
         <h3 class="api-name" id="getInstance()">getInstance</h3>
         <pre class="api-signature no-pretty-print">java-static&nbsp;fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html#getInstance()">getInstance</a>():&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></pre>
         <p>Returns the default <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance for the default <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>. </p>
-        <p>Returns the same instance for all invocations. If no instance exists, initializes a new instance with default settings.</p>
+        <p>Returns the same instance for all invocations. If no instance exists, initializes a new instance.</p>
         <p>The <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance.</p>
       </div>
       <div class="api-item"><a name="getInstance-com.google.firebase.FirebaseApp-"></a><a name="getinstance"></a>
         <h3 class="api-name" id="getInstance(com.google.firebase.FirebaseApp)">getInstance</h3>
         <pre class="api-signature no-pretty-print">java-static&nbsp;fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html#getInstance(com.google.firebase.FirebaseApp)">getInstance</a>(app:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a>):&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></pre>
         <p>Returns the default <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance for the provided <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>. </p>
-        <p>For a given <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>, invocation always returns the same instance. If no instance exists, initializes a new instance with default settings.</p>
+        <p>For a given <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>, invocation always returns the same instance. If no instance exists, initializes a new instance.</p>
         <p>The <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance.</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
@@ -676,7 +676,7 @@
               <tr>
                 <td><code>app:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code></td>
                 <td>
-                  <p>- The <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> instance that the returned <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance is associated with.</p>
+                  <p>The <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> instance that the returned <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance is associated with.</p>
                 </td>
               </tr>
             </tbody>
@@ -687,7 +687,7 @@
         <h3 class="api-name" id="getInstance(java.lang.String)">getInstance</h3>
         <pre class="api-signature no-pretty-print">java-static&nbsp;fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html#getInstance(java.lang.String)">getInstance</a>(database:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-string/index.html">String</a>):&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></pre>
         <p>Returns the <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance for the default <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>. </p>
-        <p>Returns the same instance for all invocations given the same database parameter. If no instance exists, initializes a new instance with default settings.</p>
+        <p>Returns the same instance for all invocations given the same database parameter. If no instance exists, initializes a new instance.</p>
         <p>The <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance.</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
@@ -704,7 +704,7 @@
               <tr>
                 <td><code>database:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-string/index.html">String</a></code></td>
                 <td>
-                  <p>- The name of database.</p>
+                  <p>The database ID.</p>
                 </td>
               </tr>
             </tbody>
@@ -715,7 +715,7 @@
         <h3 class="api-name" id="getInstance(com.google.firebase.FirebaseApp,java.lang.String)">getInstance</h3>
         <pre class="api-signature no-pretty-print">java-static&nbsp;fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html#getInstance(com.google.firebase.FirebaseApp,java.lang.String)">getInstance</a>(app:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a>,&nbsp;database:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-string/index.html">String</a>):&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></pre>
         <p>Returns the <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance for the provided <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code>. </p>
-        <p>Returns the same instance for all invocations given the same <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> and database parameter. If no instance exists, initializes a new instance with default settings.</p>
+        <p>Returns the same instance for all invocations given the same <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> and database parameter. If no instance exists, initializes a new instance.</p>
         <p>The <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance.</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
@@ -732,13 +732,13 @@
               <tr>
                 <td><code>app:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code></td>
                 <td>
-                  <p>- The <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> instance that the returned <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance is associated with.</p>
+                  <p>The <code><a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a></code> instance that the returned <code><a href="/docs/reference/kotlin/com/google/firebase/firestore/FirebaseFirestore.html">FirebaseFirestore</a></code> instance is associated with.</p>
                 </td>
               </tr>
               <tr>
                 <td><code>database:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-string/index.html">String</a></code></td>
                 <td>
-                  <p>- The name of database.</p>
+                  <p>The database ID.</p>
                 </td>
               </tr>
             </tbody>

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jul 12, 2023

Coverage Report 1

Affected Products

  • firebase-firestore

    Overall coverage changed from 44.35% (4d1cc39) to 44.33% (64ef2fc) by -0.02%.

    FilenameBase (4d1cc39)Merge (64ef2fc)Diff
    LruGarbageCollector.java97.27%93.64%-3.64%
    SetMutation.java97.22%94.44%-2.78%

Test Logs

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

@github-actions
Copy link
Contributor

github-actions bot commented Jul 12, 2023

Unit Test Results

   162 files  +   108     162 suites  +108   2m 20s ⏱️ - 4m 45s
1 164 tests +   694  1 148 ✔️ +   678  16 💤 +16  0 ±0 
2 328 runs  +1 388  2 296 ✔️ +1 356  32 💤 +32  0 ±0 

Results for commit b5d65ba. ± Comparison against base commit 4d1cc39.

This pull request removes 470 and adds 1164 tests. Note that renamed tests count towards both.
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testBindsService_oAndTargetingO
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testNoWrappedIntent
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testNullIntent
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_OTargetingO_highPriority
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_fallsBackToBindService
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO[19]
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO[21]
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO[22]
com.google.firebase.iid.FirebaseInstanceIdWithFcmReceiverRoboTest ‑ testStartsService_notOButTargetingO[23]
…
com.google.firebase.TimestampTest ‑ testCompare
com.google.firebase.TimestampTest ‑ testFromDate
com.google.firebase.TimestampTest ‑ testRejectBadDates
com.google.firebase.TimestampTest ‑ testTimestampParcelable
com.google.firebase.firestore.AggregateQuerySnapshotTest ‑ createWithCountShouldReturnInstanceWithTheGivenQueryAndCount
com.google.firebase.firestore.AggregateQueryTest ‑ testSourceMustNotBeNull
com.google.firebase.firestore.BlobTest ‑ testComparison
com.google.firebase.firestore.BlobTest ‑ testEquals
com.google.firebase.firestore.BlobTest ‑ testMutableBytes
com.google.firebase.firestore.CollectionReferenceTest ‑ testEquals
…

♻️ This comment has been updated with latest results.

@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

  • fire-fst

    DeviceStatisticsDistributions
    oriole-32
    Percentile4d1cc3964ef2fcDiffSignificant (?)
    p10325 ±19 μs314 ±12 μs-10.9 μs (-3.4%)NO
    p25340 ±21 μs325 ±17 μs-14.7 μs (-4.3%)NO
    p50360 ±32 μs343 ±27 μs-17.5 μs (-4.8%)NO
    p75395 ±59 μs381 ±74 μs-14.6 μs (-3.7%)NO
    p90452 ±117 μs434 ±115 μs-18.4 μs (-4.1%)NO

    20 test runs in comparison
    CommitTest Runs
    4d1cc39
    • 2023-07-11_20:46:15.787298_iVgi
    • 2023-07-11_20:46:15.790597_mFVb
    • 2023-07-11_20:46:15.790617_vqYw
    • 2023-07-11_20:46:15.790625_PWnY
    • 2023-07-11_20:46:15.790633_aYqd
    • 2023-07-11_20:46:15.790732_vLrc
    • 2023-07-11_20:46:15.790741_eGlR
    • 2023-07-11_20:46:15.790747_dLZh
    • 2023-07-11_20:46:15.790754_cdsX
    • 2023-07-11_20:46:15.790760_gpFg
    64ef2fc
    • 2023-07-12_15:39:28.131694_ILmw
    • 2023-07-12_15:39:28.135241_cPdh
    • 2023-07-12_15:39:28.135273_yXsn
    • 2023-07-12_15:39:28.135291_iLlf
    • 2023-07-12_15:39:28.135306_TkGD
    • 2023-07-12_15:39:28.135322_WXCv
    • 2023-07-12_15:39:28.135347_DIjJ
    • 2023-07-12_15:39:28.135362_VqNO
    • 2023-07-12_15:39:28.135377_McTB
    • 2023-07-12_15:39:28.135392_kLgd
    redfin-30
    Percentile4d1cc3964ef2fcDiffSignificant (?)
    p10607 ±26 μs609 ±40 μs+1.79 μs (+0.3%)NO
    p25623 ±33 μs625 ±42 μs+2.24 μs (+0.4%)NO
    p50649 ±39 μs647 ±45 μs-1.79 μs (-0.3%)NO
    p75691 ±56 μs684 ±51 μs-7.68 μs (-1.1%)NO
    p90749 ±85 μs727 ±63 μs-21.9 μs (-2.9%)NO

    20 test runs in comparison
    CommitTest Runs
    4d1cc39
    • 2023-07-11_20:46:15.787298_iVgi
    • 2023-07-11_20:46:15.790597_mFVb
    • 2023-07-11_20:46:15.790617_vqYw
    • 2023-07-11_20:46:15.790625_PWnY
    • 2023-07-11_20:46:15.790633_aYqd
    • 2023-07-11_20:46:15.790732_vLrc
    • 2023-07-11_20:46:15.790741_eGlR
    • 2023-07-11_20:46:15.790747_dLZh
    • 2023-07-11_20:46:15.790754_cdsX
    • 2023-07-11_20:46:15.790760_gpFg
    64ef2fc
    • 2023-07-12_15:39:28.131694_ILmw
    • 2023-07-12_15:39:28.135241_cPdh
    • 2023-07-12_15:39:28.135273_yXsn
    • 2023-07-12_15:39:28.135291_iLlf
    • 2023-07-12_15:39:28.135306_TkGD
    • 2023-07-12_15:39:28.135322_WXCv
    • 2023-07-12_15:39:28.135347_DIjJ
    • 2023-07-12_15:39:28.135362_VqNO
    • 2023-07-12_15:39:28.135377_McTB
    • 2023-07-12_15:39:28.135392_kLgd
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentile4d1cc3964ef2fcDiffSignificant (?)
    p10202 ±7 ms204 ±3 ms+1.68 ms (+0.8%)NO
    p25209 ±8 ms210 ±3 ms+506 μs (+0.2%)NO
    p50217 ±8 ms217 ±2 ms-170 μs (-0.1%)NO
    p75226 ±8 ms226 ±2 ms+286 μs (+0.1%)NO
    p90233 ±7 ms237 ±4 ms+3.57 ms (+1.5%)NO

    20 test runs in comparison
    CommitTest Runs
    4d1cc39
    • 2023-07-11_20:46:15.787298_iVgi
    • 2023-07-11_20:46:15.790597_mFVb
    • 2023-07-11_20:46:15.790617_vqYw
    • 2023-07-11_20:46:15.790625_PWnY
    • 2023-07-11_20:46:15.790633_aYqd
    • 2023-07-11_20:46:15.790732_vLrc
    • 2023-07-11_20:46:15.790741_eGlR
    • 2023-07-11_20:46:15.790747_dLZh
    • 2023-07-11_20:46:15.790754_cdsX
    • 2023-07-11_20:46:15.790760_gpFg
    64ef2fc
    • 2023-07-12_15:39:28.131694_ILmw
    • 2023-07-12_15:39:28.135241_cPdh
    • 2023-07-12_15:39:28.135273_yXsn
    • 2023-07-12_15:39:28.135291_iLlf
    • 2023-07-12_15:39:28.135306_TkGD
    • 2023-07-12_15:39:28.135322_WXCv
    • 2023-07-12_15:39:28.135347_DIjJ
    • 2023-07-12_15:39:28.135362_VqNO
    • 2023-07-12_15:39:28.135377_McTB
    • 2023-07-12_15:39:28.135392_kLgd
    redfin-30
    Percentile4d1cc3964ef2fcDiffSignificant (?)
    p10242 ±3 ms266 ±5 ms+24.3 ms (+10.1%)YES
    p25248 ±3 ms272 ±5 ms+23.3 ms (+9.4%)YES
    p50255 ±3 ms281 ±4 ms+25.8 ms (+10.1%)YES
    p75264 ±3 ms291 ±6 ms+27.6 ms (+10.5%)MAYBE
    p90273 ±4 ms310 ±11 ms+36.7 ms (+13.4%)MAYBE

    20 test runs in comparison
    CommitTest Runs
    4d1cc39
    • 2023-07-11_20:46:15.787298_iVgi
    • 2023-07-11_20:46:15.790597_mFVb
    • 2023-07-11_20:46:15.790617_vqYw
    • 2023-07-11_20:46:15.790625_PWnY
    • 2023-07-11_20:46:15.790633_aYqd
    • 2023-07-11_20:46:15.790732_vLrc
    • 2023-07-11_20:46:15.790741_eGlR
    • 2023-07-11_20:46:15.790747_dLZh
    • 2023-07-11_20:46:15.790754_cdsX
    • 2023-07-11_20:46:15.790760_gpFg
    64ef2fc
    • 2023-07-12_15:39:28.131694_ILmw
    • 2023-07-12_15:39:28.135241_cPdh
    • 2023-07-12_15:39:28.135273_yXsn
    • 2023-07-12_15:39:28.135291_iLlf
    • 2023-07-12_15:39:28.135306_TkGD
    • 2023-07-12_15:39:28.135322_WXCv
    • 2023-07-12_15:39:28.135347_DIjJ
    • 2023-07-12_15:39:28.135362_VqNO
    • 2023-07-12_15:39:28.135377_McTB
    • 2023-07-12_15:39:28.135392_kLgd

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

@tom-andersen tom-andersen merged commit c4b5a92 into master Jul 14, 2023
@tom-andersen tom-andersen deleted the tomandersen/multidbDocChanges branch July 14, 2023 18:10
@firebase firebase locked and limited conversation to collaborators Aug 14, 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.

4 participants