Skip to content

Add documentation to onMessageReceived on the time to handle a message. #5147

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 7, 2023

Conversation

gsakakihara
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2023

Javadoc Changes:
--- /Users/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/messaging/FirebaseMessagingService.html	2023-07-07 18:33:27.000000000 +0000
+++ /Users/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/messaging/FirebaseMessagingService.html	2023-07-07 18:28:31.000000000 +0000
@@ -1654,6 +1654,7 @@
         <h3 class="api-name" id="onMessageReceived(com.google.firebase.messaging.RemoteMessage)">onMessageReceived</h3>
         <pre class="api-signature no-pretty-print">@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/WorkerThread.html">WorkerThread</a><br>public&nbsp;void&nbsp;<a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessagingService.html#onMessageReceived(com.google.firebase.messaging.RemoteMessage)">onMessageReceived</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/messaging/RemoteMessage.html">RemoteMessage</a>&nbsp;message)</pre>
         <p>Called when a message is received. </p>
+        <p>This should complete within 20 seconds. Taking longer may interfere with your ability to complete your work and may affect pending messages. </p>
         <p>This is also called when a notification message is received while the app is in the foreground. The notification parameters can be retrieved with <code><a href="/docs/reference/android/com/google/firebase/messaging/RemoteMessage.html#getNotification()">getNotification</a></code>.</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
--- /Users/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/messaging/FirebaseMessagingService.html	2023-07-07 18:33:27.000000000 +0000
+++ /Users/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/messaging/FirebaseMessagingService.html	2023-07-07 18:28:31.000000000 +0000
@@ -1654,6 +1654,7 @@
         <h3 class="api-name" id="onMessageReceived(com.google.firebase.messaging.RemoteMessage)">onMessageReceived</h3>
         <pre class="api-signature no-pretty-print">@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/WorkerThread.html">WorkerThread</a><br>fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessagingService.html#onMessageReceived(com.google.firebase.messaging.RemoteMessage)">onMessageReceived</a>(message:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/messaging/RemoteMessage.html">RemoteMessage</a>):&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-unit/index.html">Unit</a></pre>
         <p>Called when a message is received. </p>
+        <p>This should complete within 20 seconds. Taking longer may interfere with your ability to complete your work and may affect pending messages. </p>
         <p>This is also called when a notification message is received while the app is in the foreground. The notification parameters can be retrieved with <code><a href="/docs/reference/kotlin/com/google/firebase/messaging/RemoteMessage.html#getNotification()">getNotification</a></code>.</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">

@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

  • firebase-messaging

    Overall coverage changed from 84.14% (593f4cd) to 84.10% (32532c3) by -0.04%.

    FilenameBase (593f4cd)Merge (32532c3)Diff
    FirebaseMessaging.java73.13%73.57%+0.44%
    MessagingAnalytics.java81.78%81.38%-0.40%
    Metadata.java41.27%36.51%-4.76%
    RequestDeduplicator.java80.00%90.00%+10.00%

Test Logs

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

@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2023

Unit Test Results

  54 files   -    108    54 suites   - 108   7m 0s ⏱️ + 4m 42s
470 tests  -    694  470 ✔️  -    678  0 💤  - 16  0 ±0 
940 runs   - 1 388  940 ✔️  - 1 356  0 💤  - 32  0 ±0 

Results for commit 92f1d35. ± Comparison against base commit 593f4cd.

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • firebase-messaging

    TypeBase (593f4cd)Merge (32532c3)Diff
    aar142 kB142 kB+5 B (+0.0%)
    apk (release)1.72 MB1.72 MB+28 B (+0.0%)

Test Logs

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

    DeviceStatisticsDistributions
    oriole-32
    Percentile593f4cd32532c3DiffSignificant (?)
    p101.55 ±2 ms1.66 ±2 ms+110 μs (+7.1%)NO
    p251.67 ±2 ms1.78 ±2 ms+110 μs (+6.6%)NO
    p501.83 ±2 ms1.96 ±2 ms+137 μs (+7.5%)NO
    p752.08 ±2 ms2.26 ±2 ms+175 μs (+8.4%)NO
    p902.41 ±2 ms2.63 ±3 ms+217 μs (+9.0%)NO

    20 test runs in comparison
    CommitTest Runs
    593f4cd
    • 2023-07-06_20:05:21.443794_NyKV
    • 2023-07-06_20:05:21.445931_bghc
    • 2023-07-06_20:05:21.445942_qGua
    • 2023-07-06_20:05:21.445948_LZcK
    • 2023-07-06_20:05:21.445954_UwlU
    • 2023-07-06_20:05:21.445960_xIcX
    • 2023-07-06_20:05:21.445965_hEYl
    • 2023-07-06_20:05:21.445970_nEBk
    • 2023-07-06_20:05:21.445976_GZVR
    • 2023-07-06_20:05:21.445984_UbnA
    32532c3
    • 2023-07-07_18:39:17.676271_swRZ
    • 2023-07-07_18:39:17.680058_kERP
    • 2023-07-07_18:39:17.680094_QcaC
    • 2023-07-07_18:39:17.680101_eYOg
    • 2023-07-07_18:39:17.680107_NvUq
    • 2023-07-07_18:39:17.680130_GNTO
    • 2023-07-07_18:39:17.680135_NRAx
    • 2023-07-07_18:39:17.680140_TbLU
    • 2023-07-07_18:39:17.680146_NeFR
    • 2023-07-07_18:39:17.680155_Bfhv
    redfin-30
    Percentile593f4cd32532c3DiffSignificant (?)
    p101.20 ±0.5 ms1.65 ±2 ms+445 μs (+37.0%)NO
    p251.33 ±0.6 ms1.90 ±2 ms+576 μs (+43.3%)NO
    p501.50 ±0.8 ms2.21 ±3 ms+717 μs (+47.9%)NO
    p751.80 ±1 ms2.74 ±3 ms+932 μs (+51.7%)NO
    p902.36 ±2 ms3.74 ±4 ms+1.38 ms (+58.6%)NO

    20 test runs in comparison
    CommitTest Runs
    593f4cd
    • 2023-07-06_20:05:21.443794_NyKV
    • 2023-07-06_20:05:21.445931_bghc
    • 2023-07-06_20:05:21.445942_qGua
    • 2023-07-06_20:05:21.445948_LZcK
    • 2023-07-06_20:05:21.445954_UwlU
    • 2023-07-06_20:05:21.445960_xIcX
    • 2023-07-06_20:05:21.445965_hEYl
    • 2023-07-06_20:05:21.445970_nEBk
    • 2023-07-06_20:05:21.445976_GZVR
    • 2023-07-06_20:05:21.445984_UbnA
    32532c3
    • 2023-07-07_18:39:17.676271_swRZ
    • 2023-07-07_18:39:17.680058_kERP
    • 2023-07-07_18:39:17.680094_QcaC
    • 2023-07-07_18:39:17.680101_eYOg
    • 2023-07-07_18:39:17.680107_NvUq
    • 2023-07-07_18:39:17.680130_GNTO
    • 2023-07-07_18:39:17.680135_NRAx
    • 2023-07-07_18:39:17.680140_TbLU
    • 2023-07-07_18:39:17.680146_NeFR
    • 2023-07-07_18:39:17.680155_Bfhv
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentile593f4cd32532c3DiffSignificant (?)
    p10199 ±3 ms203 ±3 ms+4.17 ms (+2.1%)NO
    p25205 ±3 ms209 ±3 ms+3.86 ms (+1.9%)NO
    p50212 ±4 ms217 ±2 ms+5.40 ms (+2.6%)NO
    p75220 ±4 ms227 ±4 ms+6.99 ms (+3.2%)NO
    p90228 ±5 ms239 ±4 ms+11.0 ms (+4.8%)NO

    20 test runs in comparison
    CommitTest Runs
    593f4cd
    • 2023-07-06_20:05:21.443794_NyKV
    • 2023-07-06_20:05:21.445931_bghc
    • 2023-07-06_20:05:21.445942_qGua
    • 2023-07-06_20:05:21.445948_LZcK
    • 2023-07-06_20:05:21.445954_UwlU
    • 2023-07-06_20:05:21.445960_xIcX
    • 2023-07-06_20:05:21.445965_hEYl
    • 2023-07-06_20:05:21.445970_nEBk
    • 2023-07-06_20:05:21.445976_GZVR
    • 2023-07-06_20:05:21.445984_UbnA
    32532c3
    • 2023-07-07_18:39:17.676271_swRZ
    • 2023-07-07_18:39:17.680058_kERP
    • 2023-07-07_18:39:17.680094_QcaC
    • 2023-07-07_18:39:17.680101_eYOg
    • 2023-07-07_18:39:17.680107_NvUq
    • 2023-07-07_18:39:17.680130_GNTO
    • 2023-07-07_18:39:17.680135_NRAx
    • 2023-07-07_18:39:17.680140_TbLU
    • 2023-07-07_18:39:17.680146_NeFR
    • 2023-07-07_18:39:17.680155_Bfhv
    redfin-30
    Percentile593f4cd32532c3DiffSignificant (?)
    p10244 ±3 ms265 ±3 ms+21.1 ms (+8.7%)YES
    p25250 ±4 ms271 ±2 ms+21.4 ms (+8.6%)YES
    p50257 ±5 ms280 ±3 ms+22.4 ms (+8.7%)MAYBE
    p75266 ±5 ms288 ±4 ms+22.6 ms (+8.5%)MAYBE
    p90275 ±6 ms301 ±6 ms+26.0 ms (+9.5%)MAYBE

    20 test runs in comparison
    CommitTest Runs
    593f4cd
    • 2023-07-06_20:05:21.443794_NyKV
    • 2023-07-06_20:05:21.445931_bghc
    • 2023-07-06_20:05:21.445942_qGua
    • 2023-07-06_20:05:21.445948_LZcK
    • 2023-07-06_20:05:21.445954_UwlU
    • 2023-07-06_20:05:21.445960_xIcX
    • 2023-07-06_20:05:21.445965_hEYl
    • 2023-07-06_20:05:21.445970_nEBk
    • 2023-07-06_20:05:21.445976_GZVR
    • 2023-07-06_20:05:21.445984_UbnA
    32532c3
    • 2023-07-07_18:39:17.676271_swRZ
    • 2023-07-07_18:39:17.680058_kERP
    • 2023-07-07_18:39:17.680094_QcaC
    • 2023-07-07_18:39:17.680101_eYOg
    • 2023-07-07_18:39:17.680107_NvUq
    • 2023-07-07_18:39:17.680130_GNTO
    • 2023-07-07_18:39:17.680135_NRAx
    • 2023-07-07_18:39:17.680140_TbLU
    • 2023-07-07_18:39:17.680146_NeFR
    • 2023-07-07_18:39:17.680155_Bfhv

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

@gsakakihara gsakakihara merged commit d41d2ab into master Jul 7, 2023
@gsakakihara gsakakihara deleted the gsakakihara/timeout_documentation branch July 7, 2023 18:59
@firebase firebase locked and limited conversation to collaborators Aug 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants