8000 Support for java.time.Instant in Firebase Timestamp by cherylEnkidu · Pull Request #5853 · firebase/firebase-android-sdk · GitHub
[go: up one dir, main page]

Skip to content

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>
<
8000
span class="pl-mdr">@@ -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
@cherylEnkidu cherylEnkidu requested a review from daymxn April 10, 2024 22:40
@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.

Co-authored-by: Daymon <17409137+daymxn@users.noreply.github.com>
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
@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
0