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

Auto-Instrumentation UI thread #36

Merged
merged 9 commits into from
Dec 15, 2022
Merged

Auto-Instrumentation UI thread #36

merged 9 commits into from
Dec 15, 2022

Conversation

romtsn
Copy link
Member

@romtsn romtsn commented Nov 9, 2022

Add new blocked_main_thread attribute to the data key-value map of the Span interface, indicating whether an auto-instrumented span has run its entire duration on the UI/Main thread.

Rendered RFC

@romtsn romtsn changed the title File I/O on main thread Auto-Instrumentation UI thread Nov 9, 2022
@romtsn romtsn marked this pull request as ready for review November 9, 2022 13:22
@marandaneto
Copy link
Contributor

Since somebody asked if this info should go under the transaction or span, I believe the span makes more sense.
Also, consider this: getsentry/team-mobile#36 (comment)
Knowing if the app is in the background or foreground also makes a lot of sense, and this one should go under the transaction/event.
Both are equally important and somehow related, we don't need to address the later in this spec, but we can follow up later.

Co-authored-by: Dhiogo Brustolin <dhiogorb@gmail.com>
markushi added a commit to getsentry/relay that referenced this pull request Nov 28, 2022
In order to be consistent with getsentry/rfcs#36
wmak added a commit to getsentry/sentry that referenced this pull request Nov 28, 2022
- Introduce the file io on main thread performance problem. Not hooking
up the options or anything yet. Just want to get the main core of the
code ready to go first
- I think this won't actually create any of the file io perf problems
yet since the option is only added to default.py with 0
- Depends on getsentry/rfcs#36
Copy link
Contributor

@brustolin brustolin left a comment

Choose a reason for hiding this comment

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

This looks very good!

@romtsn romtsn merged commit 64fd295 into main Dec 15, 2022
@romtsn romtsn deleted the feat/file-io-main-thread branch December 15, 2022 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants