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

Support for java.time.Instant in Firebase Timestamp #5853

Merged
merged 18 commits into from
Apr 16, 2024

Conversation

cherylEnkidu
Copy link
Contributor

No description provided.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Apr 10, 2024

Coverage Report 1

Affected Products

  • firebase-common

    Overall coverage changed from 52.58% (ac8b196) to 52.66% (7692833) by +0.07%.

    FilenameBase (ac8b196)Merge (7692833)Diff
    Timestamp.kt76.32%77.50%+1.18%
  • firebase-firestore

    FilenameBase (ac8b196)Merge (7692833)Diff
    DeleteMutation.java95.24%90.48%-4.76%
    PatchMutation.java98.39%100.00%+1.61%

Test Logs

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

Copy link
Contributor

github-actions bot commented Apr 10, 2024

Javadoc Changes:
--- /home/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/Timestamp.html	2024-04-16 21:33:17.906582503 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/Timestamp.html	2024-04-16 21:30:35.843596710 +0000
@@ -107,6 +107,11 @@
           </tr>
           <tr>
             <td>
+              <div><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/RequiresApi.html">RequiresApi</a>(value&nbsp;=&nbsp;26)<br><a href="/docs/reference/android/com/google/firebase/Timestamp.html#Timestamp(java.time.Instant)">Timestamp</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a>&nbsp;time)</code></div>
+            </td>
+          </tr>
+          <tr>
+            <td>
               <div><code><a href="/docs/reference/android/com/google/firebase/Timestamp.html#Timestamp(kotlin.Long,kotlin.Int)">Timestamp</a>(long&nbsp;seconds,&nbsp;int&nbsp;nanoseconds)</code></div>
               <p>Creates a new <code><a href="/docs/reference/android/com/google/firebase/Timestamp.html">Timestamp</a></code>.</p>
             </td>
@@ -164,6 +169,13 @@
             </td>
           </tr>
           <tr>
+            <td><code>final @<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a></code></td>
+            <td>
+              <div><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/RequiresApi.html">RequiresApi</a>(value&nbsp;=&nbsp;26)<br><a href="/docs/reference/android/com/google/firebase/Timestamp.html#toInstant()">toInstant</a>()</code></div>
+              <p>Returns a new <code><a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a></code> that matches the time defined by this timestamp.</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></code></td>
             <td>
               <div><code><a href="/docs/reference/android/com/google/firebase/Timestamp.html#toString()">toString</a>()</code></div>
@@ -199,6 +211,10 @@
         <h3 class="api-name" id="Timestamp(java.util.Date)">Timestamp</h3>
         <pre class="api-signature no-pretty-print">public&nbsp;<a href="/docs/reference/android/com/google/firebase/Timestamp.html#Timestamp(java.util.Date)">Timestamp</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/util/Date.html">Date</a>&nbsp;date)</pre>
       </div>
+      <div class="api-item"><a name="Timestamp-java.time.Instant-"></a><a name="timestamp"></a>
+        <h3 class="api-name" id="Timestamp(java.time.Instant)">Timestamp</h3>
+        <pre class="api-signature no-pretty-print">@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/RequiresApi.html">RequiresApi</a>(value&nbsp;=&nbsp;26)<br>public&nbsp;<a href="/docs/reference/android/com/google/firebase/Timestamp.html#Timestamp(java.time.Instant)">Timestamp</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a>&nbsp;time)</pre>
+      </div>
       <div class="api-item"><a name="Timestamp(kotlin.Long, kotlin.Int)"></a><a name="Timestamp-kotlin.Long-kotlin.Int-"></a><a name="timestamp"></a>
         <h3 class="api-name" id="Timestamp(kotlin.Long,kotlin.Int)">Timestamp</h3>
         <pre class="api-signature no-pretty-print">public&nbsp;<a href="/docs/reference/android/com/google/firebase/Timestamp.html#Timestamp(kotlin.Long,kotlin.Int)">Timestamp</a>(long&nbsp;seconds,&nbsp;int&nbsp;nanoseconds)</pre>
@@ -260,6 +276,11 @@
         <p>Returns a new <code><a href="https://developer.android.com/reference/kotlin/java/util/Date.html">Date</a></code> corresponding to this timestamp.</p>
         <p>This may lose precision.</p>
       </div>
+      <div class="api-item"><a name="toInstant--"></a><a name="toinstant"></a>
+        <h3 class="api-name" id="toInstant()">toInstant</h3>
+        <pre class="api-signature no-pretty-print">@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/RequiresApi.html">RequiresApi</a>(value&nbsp;=&nbsp;26)<br>public&nbsp;final&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/Timestamp.html#toInstant()">toInstant</a>()</pre>
+        <p>Returns a new <code><a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a></code> that matches the time defined by this timestamp.</p>
+      </div>
       <div class="api-item"><a name="toString--"></a><a name="tostring"></a>
         <h3 class="api-name" id="toString()">toString</h3>
         <pre class="api-signature no-pretty-print">public&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;<a href="/docs/reference/android/com/google/firebase/Timestamp.html#toString()">toString</a>()</pre>
--- /home/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/Timestamp.html	2024-04-16 21:33:17.906582503 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/Timestamp.html	2024-04-16 21:30:35.839596736 +0000
@@ -96,6 +96,11 @@
           </tr>
           <tr>
             <td>
+              <div><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/RequiresApi.html">RequiresApi</a>(value&nbsp;=&nbsp;26)<br><a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#Timestamp(java.time.Instant)">Timestamp</a>(time:&nbsp;<a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a>)</code></div>
+            </td>
+          </tr>
+          <tr>
+            <td>
               <div><code><a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#Timestamp(kotlin.Long,kotlin.Int)">Timestamp</a>(seconds:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-long/index.html">Long</a>,&nbsp;nanoseconds:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-int/index.html">Int</a>)</code></div>
               <p>Creates a new <code><a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html">Timestamp</a></code>.</p>
             </td>
@@ -147,6 +152,13 @@
             </td>
           </tr>
           <tr>
+            <td><code><a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a></code></td>
+            <td>
+              <div><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/RequiresApi.html">RequiresApi</a>(value&nbsp;=&nbsp;26)<br><a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#toInstant()">toInstant</a>()</code></div>
+              <p>Returns a new <code><a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a></code> that matches the time defined by this timestamp.</p>
+            </td>
+          </tr>
+          <tr>
             <td><code>open <a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-string/index.html">String</a></code></td>
             <td>
               <div><code><a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#toString()">toString</a>()</code></div>
@@ -208,6 +220,10 @@
         <h3 class="api-name" id="Timestamp(java.util.Date)">Timestamp</h3>
         <pre class="api-signature no-pretty-print"><a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#Timestamp(java.util.Date)">Timestamp</a>(date:&nbsp;<a href="https://developer.android.com/reference/kotlin/java/util/Date.html">Date</a>)</pre>
       </div>
+      <div class="api-item"><a name="Timestamp-java.time.Instant-"></a><a name="timestamp"></a>
+        <h3 class="api-name" id="Timestamp(java.time.Instant)">Timestamp</h3>
+        <pre class="api-signature no-pretty-print">@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/RequiresApi.html">RequiresApi</a>(value&nbsp;=&nbsp;26)<br><a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#Timestamp(java.time.Instant)">Timestamp</a>(time:&nbsp;<a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a>)</pre>
+      </div>
       <div class="api-item"><a name="Timestamp(kotlin.Long, kotlin.Int)"></a><a name="Timestamp-kotlin.Long-kotlin.Int-"></a><a name="timestamp"></a>
         <h3 class="api-name" id="Timestamp(kotlin.Long,kotlin.Int)">Timestamp</h3>
         <pre class="api-signature no-pretty-print"><a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#Timestamp(kotlin.Long,kotlin.Int)">Timestamp</a>(seconds:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-long/index.html">Long</a>,&nbsp;nanoseconds:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-int/index.html">Int</a>)</pre>
@@ -265,6 +281,11 @@
         <p>Returns a new <code><a href="https://developer.android.com/reference/kotlin/java/util/Date.html">Date</a></code> corresponding to this timestamp.</p>
         <p>This may lose precision.</p>
       </div>
+      <div class="api-item"><a name="toInstant--"></a><a name="toinstant"></a>
+        <h3 class="api-name" id="toInstant()">toInstant</h3>
+        <pre class="api-signature no-pretty-print">@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/RequiresApi.html">RequiresApi</a>(value&nbsp;=&nbsp;26)<br>fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#toInstant()">toInstant</a>():&nbsp;<a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a></pre>
+        <p>Returns a new <code><a href="https://developer.android.com/reference/kotlin/java/time/Instant.html">Instant</a></code> that matches the time defined by this timestamp.</p>
+      </div>
       <div class="api-item"><a name="toString--"></a><a name="tostring"></a>
         <h3 class="api-name" id="toString()">toString</h3>
         <pre class="api-signature no-pretty-print">open&nbsp;fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/Timestamp.html#toString()">toString</a>():&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-string/index.html">String</a></pre>

Copy link
Contributor

github-actions bot commented Apr 10, 2024

Unit Test Results

   220 files  ±0     220 suites  ±0   4m 11s ⏱️ -8s
1 301 tests +1  1 285 ✔️ +1  16 💤 ±0  0 ±0 
2 618 runs  +2  2 586 ✔️ +2  32 💤 ±0  0 ±0 

Results for commit 127de21. ± Comparison against base commit ac8b196.

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Apr 10, 2024

Size Report 1

Affected Products

  • firebase-common

    TypeBase (ac8b196)Merge (7692833)Diff
    aar115 kB116 kB+310 B (+0.3%)
    apk (release)4.49 MB4.49 MB+356 B (+0.0%)
  • firebase-firestore

    TypeBase (ac8b196)Merge (7692833)Diff
    apk (release)11.3 MB11.3 MB+372 B (+0.0%)

Test Logs

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

@firebase firebase deleted a comment from google-oss-bot Apr 10, 2024
@firebase firebase deleted a comment from google-oss-bot Apr 10, 2024
@firebase firebase deleted a comment from google-oss-bot Apr 10, 2024
@firebase firebase deleted a comment from google-oss-bot Apr 10, 2024
Copy link
Contributor

@dconeybe dconeybe left a comment

Choose a reason for hiding this comment

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

LGTM. I'll mark my review as "Approved" once the API proposal (go/java-instant-in-firebase-timestamp for Googlers) is approved by API council.

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.

LGTM, I defer the final approval to @dconeybe for post api council.

firebase-firestore/CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

@dconeybe dconeybe left a comment

Choose a reason for hiding this comment

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

Approved! Please wait for the base PR to be merged so you can merge this PR directly into the master branch.

Base automatically changed from daymon-refactor-timestamp to master April 16, 2024 20:55
@google-oss-bot
Copy link
Contributor

The public api surface has changed for the subproject firebase-common:
error: Added constructor com.google.firebase.Timestamp(java.time.Instant) [AddedMethod]
error: Added method com.google.firebase.Timestamp.toInstant() [AddedMethod]

Please update the api.txt files for the subprojects being affected by this change by running ./gradlew ${subproject}:generateApiTxtFile. Also perform a major/minor bump accordingly.

@firebase firebase deleted a comment from google-oss-bot Apr 16, 2024
@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
    Percentileac8b1967692833DiffSignificant (?)
    p10984 ±71 μs983 ±77 μs-843 ns (-0.1%)NO
    p251.07 ±0.07 ms1.07 ±0.09 ms+6.18 μs (+0.6%)NO
    p501.19 ±0.06 ms1.20 ±0.08 ms+15.4 μs (+1.3%)NO
    p751.39 ±0.08 ms1.41 ±0.09 ms+21.1 μs (+1.5%)NO
    p901.70 ±0.1 ms1.69 ±0.1 ms-7.93 μs (-0.5%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
    redfin-30
    Percentileac8b1967692833DiffSignificant (?)
    p10854 ±22 μs861 ±20 μs+7.29 μs (+0.9%)NO
    p25892 ±21 μs895 ±20 μs+3.10 μs (+0.3%)NO
    p50964 ±31 μs977 ±29 μs+13.2 μs (+1.4%)NO
    p751.26 ±0.1 ms1.25 ±0.1 ms-11.9 μs (-0.9%)NO
    p901.75 ±0.2 ms1.68 ±0.2 ms-63.9 μs (-3.7%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
  • Firebase

    DeviceStatisticsDistributions
    oriole-32
    Percentileac8b1967692833DiffSignificant (?)
    p105.64 ±0.2 ms5.86 ±0.4 ms+219 μs (+3.9%)NO
    p255.93 ±0.2 ms6.11 ±0.5 ms+184 μs (+3.1%)NO
    p506.34 ±0.2 ms6.59 ±0.7 ms+251 μs (+4.0%)NO
    p756.96 ±0.3 ms7.19 ±0.8 ms+235 μs (+3.4%)NO
    p907.82 ±0.3 ms7.98 ±0.9 ms+169 μs (+2.2%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
    redfin-30
    Percentileac8b1967692833DiffSignificant (?)
    p108.79 ±0.2 ms8.70 ±0.1 ms-90.5 μs (-1.0%)NO
    p259.05 ±0.2 ms8.95 ±0.09 ms-97.8 μs (-1.1%)NO
    p509.49 ±0.2 ms9.35 ±0.2 ms-135 μs (-1.4%)NO
    p7510.4 ±0.2 ms10.3 ±0.3 ms-42.5 μs (-0.4%)NO
    p9011.8 ±0.4 ms11.6 ±0.3 ms-165 μs (-1.4%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
  • Runtime

    DeviceStatisticsDistributions
    oriole-32
    Percentileac8b1967692833DiffSignificant (?)
    p104.53 ±0.2 ms4.73 ±0.4 ms+206 μs (+4.6%)NO
    p254.73 ±0.2 ms4.91 ±0.4 ms+177 μs (+3.7%)NO
    p505.05 ±0.2 ms5.26 ±0.6 ms+209 μs (+4.1%)NO
    p755.53 ±0.2 ms5.76 ±0.7 ms+229 μs (+4.1%)NO
    p906.26 ±0.3 ms6.43 ±0.9 ms+164 μs (+2.6%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
    redfin-30
    Percentileac8b1967692833DiffSignificant (?)
    p107.77 ±0.2 ms7.66 ±0.07 ms-118 μs (-1.5%)NO
    p257.98 ±0.1 ms7.87 ±0.1 ms-113 μs (-1.4%)NO
    p508.31 ±0.1 ms8.21 ±0.1 ms-101 μs (-1.2%)NO
    p758.96 ±0.1 ms8.81 ±0.2 ms-145 μs (-1.6%)NO
    p9010.0 ±0.3 ms9.78 ±0.4 ms-265 μs (-2.6%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
  • fire-fst

    DeviceStatisticsDistributions
    oriole-32
    Percentileac8b1967692833DiffSignificant (?)
    p10411 ±194 μs421 ±208 μs+10.1 μs (+2.4%)NO
    p25427 ±202 μs438 ±211 μs+11.7 μs (+2.7%)NO
    p50449 ±202 μs470 ±209 μs+20.9 μs (+4.7%)NO
    p75498 ±200 μs527 ±199 μs+29.0 μs (+5.8%)NO
    p90585 ±196 μs616 ±181 μs+31.3 μs (+5.4%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
    redfin-30
    Percentileac8b1967692833DiffSignificant (?)
    p10700 ±227 μs609 ±22 μs-91.7 μs (-13.1%)NO
    p25717 ±226 μs629 ±25 μs-87.6 μs (-12.2%)NO
    p50741 ±225 μs658 ±31 μs-83.4 μs (-11.3%)NO
    p75773 ±223 μs697 ±53 μs-76.8 μs (-9.9%)NO
    p90816 ±219 μs746 ±81 μs-70.2 μs (-8.6%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentileac8b1967692833DiffSignificant (?)
    p10201 ±4 ms211 ±10 ms+9.70 ms (+4.8%)NO
    p25208 ±4 ms216 ±12 ms+8.61 ms (+4.1%)NO
    p50215 ±5 ms224 ±14 ms+9.24 ms (+4.3%)NO
    p75223 ±6 ms235 ±21 ms+11.7 ms (+5.3%)NO
    p90232 ±5 ms243 ±23 ms+11.3 ms (+4.9%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm
    redfin-30
    Percentileac8b1967692833DiffSignificant (?)
    p10245 ±2 ms265 ±8 ms+19.8 ms (+8.1%)NO
    p25251 ±2 ms272 ±7 ms+20.6 ms (+8.2%)MAYBE
    p50257 ±3 ms278 ±8 ms+20.9 ms (+8.1%)NO
    p75264 ±3 ms286 ±9 ms+22.4 ms (+8.5%)NO
    p90271 ±4 ms295 ±9.6 ms+24.0 ms (+8.9%)NO

    20 test runs in comparison
    CommitTest Runs
    ac8b196
    • 2024-04-16_21:07:24.499119_txAT
    • 2024-04-16_21:07:24.499155_douf
    • 2024-04-16_21:07:24.499162_UYHK
    • 2024-04-16_21:07:24.499167_pxSt
    • 2024-04-16_21:07:24.499173_OXKr
    • 2024-04-16_21:07:24.499182_hHKs
    • 2024-04-16_21:07:24.499191_KtGC
    • 2024-04-16_21:07:24.499198_YyWL
    • 2024-04-16_21:07:24.499206_JrPN
    • 2024-04-16_21:07:24.499213_lgso
    7692833
    • 2024-04-16_21:40:26.253753_ReuT
    • 2024-04-16_21:40:26.253794_tfRB
    • 2024-04-16_21:40:26.253804_gwwV
    • 2024-04-16_21:40:26.253812_Posd
    • 2024-04-16_21:40:26.253819_PMfz
    • 2024-04-16_21:40:26.253826_VJMV
    • 2024-04-16_21:40:26.253833_fvZV
    • 2024-04-16_21:40:26.253840_FGzi
    • 2024-04-16_21:40:26.253848_xoXe
    • 2024-04-16_21:40:26.253856_MMSm

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

@cherylEnkidu cherylEnkidu merged commit 1d717ea into master Apr 16, 2024
26 of 27 checks passed
@cherylEnkidu cherylEnkidu deleted the cheryllin/Instant branch April 16, 2024 22:07
@firebase firebase locked and limited conversation to collaborators May 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants