Track app launch time #110
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Closed
LikeTheSalad added a commit that referenced this pull request Feb 6, 2023
* Added test for launch time metric * Created app launch time tracking tools * Sending app launch time metric from activity callback * Registering provider and adding logs * Adding logs * Registering launch time activity callback * Validating launch time metric test * Flushing launch time metric right away * Removing log when starting time * Sending launch time metric as milliseconds * Preferring delta aggregation for common metrics exporter * Increased android ntp root delay max * Sending launch time as gauge metric * Adding multiple metric flushing test * Fix typo in tests * Added test to verify that only the first activity tracks the startup time * Adding comments * Updated logs * Updated logs * Flushing logs for crashes using the Flusher delegator * Moving app launch time tests to their own class and resetting static state for them * Reorganizing test packages
LikeTheSalad added a commit that referenced this pull request Feb 6, 2023
* Removing session id provider config setter * Removing session id config guide from docs * Updated tests * release: define buildkite pipeline (#109) * release dry run report logs to google bucket (#115) * release: use new vault secrets (#116) * release: upload artifacts (#117) * Track app launch time (#110) * Added test for launch time metric * Created app launch time tracking tools * Sending app launch time metric from activity callback * Registering provider and adding logs * Adding logs * Registering launch time activity callback * Validating launch time metric test * Flushing launch time metric right away * Removing log when starting time * Sending launch time metric as milliseconds * Preferring delta aggregation for common metrics exporter * Increased android ntp root delay max * Sending launch time as gauge metric * Adding multiple metric flushing test * Fix typo in tests * Added test to verify that only the first activity tracks the startup time * Adding comments * Updated logs * Updated logs * Flushing logs for crashes using the Flusher delegator * Moving app launch time tests to their own class and resetting static state for them * Reorganizing test packages * Adding http status code to spans (#113) * Reorganizing okhttp event listener common code * Setting http status code to http tracked span * Added tests to verify result code in okhttp spans * Added verification for context store clearing --------- Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Closes #95