-
Notifications
You must be signed in to change notification settings - Fork 625
Add different default sampling rates for when RC fetch failed #5059
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
Conversation
Release note changesThe following release notes were modified. Please ensure they look correct. Release Notesfirebase-perf### {{perfmon}} version 20.3.3 {: #performance_v20-3-3}
* {{fixed}} Fixed app start trace creation where some measured time could be NULL GitHub [#4730](//github.com/firebase/firebase-android-sdk/issues/4730){: .external}.
* {{changed}} Adjusted default behavior when remote config fetch fails.
#### {{perfmon}} Kotlin extensions version 20.3.3 {: #performance-ktx_v20-3-3}
The Kotlin extensions library transitively includes the updated
`firebase-performance` library. The Kotlin extensions library has no additional
updates. |
Coverage Report 1Affected Products
Test Logs |
Size Report 1Affected Products
Test Logs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good to me.
I'm just thinking if the CPU and memory capturing frequency can also be reduced incase of RC fetch failures. Example: instead of capturing every 100ms, we could push the limits to once in every 300ms(??).
firebase-perf/src/main/java/com/google/firebase/perf/config/ConfigResolver.java
Show resolved
Hide resolved
Startup Time Report 1Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS. Notes
Startup Times
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for making those changes.
firebase-perf/src/main/java/com/google/firebase/perf/config/ConfigurationConstants.java
Show resolved
Hide resolved
firebase-perf/src/test/java/com/google/firebase/perf/config/ConfigurationConstantsTest.java
Show resolved
Hide resolved
* Add different defaults for when RC fetch failed * Add changelog entry * Format * Add throttled case to isLastFetchFailed check * Add rc fetch failed defaults for cpu and memory capturing frequency
No description provided.