Reduce batch size when marking articles read (#155)

jocmp · Jul 10, 2024 · 9387690 · 9387690
1 parent 3b39404
commit 9387690
Showing 1 changed file with 17 additions and 9 deletions.
diff --git a/app/src/main/java/com/capyreader/app/sync/Sync.kt b/app/src/main/java/com/capyreader/app/sync/Sync.kt
@@ -33,15 +33,23 @@ fun removeStarAsync(articleID: String, context: Context) {
  * Batch enqueue IDs to stay under 10KB work manager limit
  * https://developer.android.com/jetpack/androidx/releases/work#1.0.0-alpha08
  */
-fun markReadAsync(articleIDs: List<String>, context: Context) {
-    articleIDs.chunked(500) { batch ->
-        val data = Data
-            .Builder()
-            .putStringArray(ReadSyncWorker.ARTICLES_KEY, batch.toTypedArray())
-            .putBoolean(ReadSyncWorker.MARK_READ_KEY, true)
-            .build()
-
-        queueReadWorker(batch, data, context)
+fun markReadAsync(articleIDs: List<String>, context: Context, batchSize: Int = 100) {
+    if (batchSize < 1) {
+        return
+    }
+
+    articleIDs.chunked(batchSize) { batch ->
+        try {
+            val data = Data
+                .Builder()
+                .putStringArray(ReadSyncWorker.ARTICLES_KEY, batch.toTypedArray())
+                .putBoolean(ReadSyncWorker.MARK_READ_KEY, true)
+                .build()
+
+            queueReadWorker(batch, data, context)
+        } catch (e: IllegalStateException) {
+            markReadAsync(context = context, articleIDs = batch, batchSize = batchSize / 2)
+        }
     }
 }