-
Notifications
You must be signed in to change notification settings - Fork 37
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
Track low memory events #283
Comments
It's worth adding a min. level configuration since not all levels are critical. |
I think this is a great idea. Contributions for this are very much welcome! |
I think this is a great idea as well! We can monitor when the app is running on low memory and identify the exact time when it happens. However, I'm still not sure where to put this event in the context of OpenTelemetry Signals. We can put the low memory event as: I'm leaning towards using a Span Event, since we can get the full picture of the app's events by looking at Spans in the Trace. What do you guys think? |
What signal is appropriate depends entirely on how sessions or being modelled in the app/library that is capturing the telemetry. For instance, in the Embrace SDK, this will be SpanEvent attached to the specific span we use to model a session, but this may not be appropriate for everyone. The new In the context of this project, @breedx-splk probably has the best insight as to how best to model it, either now or in the future. |
Yeah, I agree with @bidetofevil that I believe the latest core java sdk release has the updated API that matches changes to the spec and semconv, and will allow rich objects in event payload bodies. Would be great to see what an implementation of these would look like, with respect to low memory. |
Cool! I just found out that
Agree that |
Android's ComponentCallback2 interface allows for the app to listen for OS-level memory events that tell the app the process is trying to recover memory. Surfacing these warnings/alerts (as Events?) could be useful as it could be an indication the app is running low on resources so performance could be hampered.
The text was updated successfully, but these errors were encountered: