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.

Sign up for GitHub

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

Jump to bottom

Reduce batch size when marking articles read #155

Merged

jocmp merged 1 commit into main from jc/152/crash-on-mark-read

Jul 10, 2024

Owner

jocmp commented Jul 10, 2024 •

edited

Loading

I was able to recreate #152 on a Pixel 3a. The issue is caused by the 10KB argument limit when enqueuing background jobs.

The potential fix is two parts:

Lower the batch size from 500 to 100
Retry the worker until 1 ID (url) is enqueued at a time.


          Reduce batch size when marking articles read

d2e602f

jocmp merged commit 9387690 into main

1 check passed

jocmp deleted the jc/152/crash-on-mark-read branch

July 10, 2024 02:17

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

Labels

None yet