-
Notifications
You must be signed in to change notification settings - Fork 258
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
IllegalStateException: Couldn't read row 0, col 0 from CursorWindow (in GreenDAO's list method) #413
Comments
Seems to be the same as greenrobot/greenDAO#200. |
Two more reports.
|
Application version: 32.
|
Error reports like in #566 are still coming (from one user). |
Maybe this size limit is a hint https://stackoverflow.com/a/21432966/709697 if greendao relies on android sqlite |
Yup. |
I have another idea. We always increase the offset and set it here: https://github.com/wallabag/android-app/blob/2.0.1/app/src/main/java/fr/gaulupeau/apps/Poche/service/SecondaryService.java#L312 so what happens when offset is greater than totalNumber and we do this: https://github.com/wallabag/android-app/blob/2.0.1/app/src/main/java/fr/gaulupeau/apps/Poche/service/SecondaryService.java#L280 ? |
This shouldn't cause errors. It is highly unlikely that such an error exists in SQLite implementation Android uses (and nobody knows about it). |
Yes, absolutely makes sense. Just tested it
|
Maybe reducing the query limit to 10 helps that particular user? (And does not hurt us.) |
The error happens not only for images fetching. And we can't decrease values forever. |
What's the current status of this issue? |
I think the source of the problem is the size of specific articles (so they don't fit in the cursor window), not a total number of articles, but that's just a guess. Can you give some details about the time when the error happens? If it happens in the lists, does it happen in some specific (reproducible) moment? I though about moving article content to file storage: one small DB with metadata and a bunch of files (one for each article). I haven't done any work on it yet. |
I've just started using this. |
If you have time to experiment, you can try to add around 30 small articles and check whether the app fails to show the main screen (30 is the number of articles loaded in a list by default). However, the unread and favorite tabs may be loaded too (even if you didn't open them yet), so you may need to mark all these 30 articles as favorite and add another 30 archived articles. If this will work, that would mean that it indeed breaks not because of the total number of articles. |
@drakeet thanks for sharing your solution. I'll consider applying it as a temporary measure. |
What is @drakeet's solution? (Just out of curiosity.) |
It was pretty much this code. However, be advised that this is a work-around, not a solution. |
I got the same issue that because of the cursor return row from the database with 2MB maximum |
Apparently Some experimenting needs to be done (to reproduce in dev environment). |
From google play dev console:
Application version: 29.
Android version: Android 4.4.
The text was updated successfully, but these errors were encountered: