-
-
Notifications
You must be signed in to change notification settings - Fork 316
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
fix: Detecting ANRs as false OOMs #1695
Conversation
The OOM logic falsely detected ANRs as OOMs. This is fixed now by adding a tracker for ANRs, which doesn't report ANR events to Sentry yet. Fixes GH-1645
Codecov Report
@@ Coverage Diff @@
## master #1695 +/- ##
==========================================
+ Coverage 94.70% 94.77% +0.06%
==========================================
Files 162 167 +5
Lines 7442 7579 +137
==========================================
+ Hits 7048 7183 +135
- Misses 394 396 +2
Continue to review full report at Codecov.
|
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.
8/42 files
Co-authored-by: Bruno Garcia <bruno@brunogarcia.com>
{ | ||
@synchronized(self) { | ||
SentryAppState *appState = [self.fileManager readAppState]; | ||
if (nil != appState) { |
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.
What about the first run? Will readAppState create an empty file for us on read?
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.
As we currently only use this for creating OOMs and we use it to avoid creating false OOMs, we don't need to worry about this right now, as we only create an OOM if there is an app state.
Co-authored-by: Bruno Garcia <bruno@brunogarcia.com>
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.
Great!! Let merge this!! 🚀
📜 Description
The OOM logic falsely detected ANRs as OOMs. This is fixed now by
adding a tracker for ANRs, which doesn't report ANR events to
Sentry yet.
💡 Motivation and Context
Fixes GH-1645
💚 How did you test it?
Unit tests, simulator, and on an iPhone.
📝 Checklist
🔮 Next steps
Docs PR getsentry/sentry-docs#4837