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

Possible fix to TransactionTooLarge crash #5698

Merged
merged 1 commit into from
Apr 24, 2017

Conversation

oguzkocer
Copy link
Contributor

Fixes #5456. I have been unable to reproduce this crash no matter how many pages of notifications/reader posts I have browsed. I have tested on both emulator and my Pixel, the state seems to be saved without any issues. Since the report doesn't contain much info, this is just a possible fix. I was inclined to do the same for
ReaderPostPagerActivity in case the issue is happening there, but this might be better to identify the issue. As @maxme mentioned, the issue started after NotificationDetailFragmentAdapter, so that should be the first place we test.

To test:

  • No idea, since I can't reproduce the issue myself

/cc @maxme

@oguzkocer oguzkocer added this to the 7.2 milestone Apr 19, 2017
@maxme maxme self-assigned this Apr 21, 2017
@maxme
Copy link
Contributor

maxme commented Apr 24, 2017

:shipit: we'll see if that fixes the issue in 7.2.

@maxme maxme merged commit 44dae1d into release/7.2 Apr 24, 2017
@maxme maxme deleted the issue/transaction-too-large-crash branch April 24, 2017 06:34
@gaurav-mishra8
Copy link

@maxme i have been facing the same crash and wanted to know if the commit fixed this issue for you guys ?
We tried same patch but it started giving NPE at bundle.putParcelableArray("states", null)
as the super.saveState() returned a null bundle.

@mzorz
Copy link
Contributor

mzorz commented Mar 5, 2018

@gaurav-mishra8 unfortunately that hasn't been the case - it's still there and we haven't found a consistent fix for it, other than palliatives or workarounds

@maxme
Copy link
Contributor

maxme commented Mar 5, 2018

it's still there and we haven't found a consistent fix for it, other than palliatives or workarounds

There is a known issue that must be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants