fix: rebuilding tree error when the page number is large enough in sqlite (#2830)

When the total page number is large enough (usually about 80+), sqlite will throw error: "Too many variables". This commit reduces the chunk size for sqlite configuration.

fix: rebuilding tree error when the page number is large enough in sqlite (#2830)
4b80bab8 · scienceasdf · GitHub · 52304a81 · 4b80bab8
Unverified Commit 4b80bab8 authored Jan 30, 2021 by scienceasdf Committed by GitHub Jan 29, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 2 deletions

rebuild-tree.js server/jobs/rebuild-tree.js +8 -2

No files found.
--- a/server/jobs/rebuild-tree.js
+++ b/server/jobs/rebuild-tree.js
@@ -57,8 +57,14 @@ module.exports = async (pageId) => {
    await WIKI.models.knex.table('pageTree').truncate()
    if (tree.length > 0) {
      // -> Save in chunks, because of per query max parameters (35k Postgres, 2k MSSQL, 1k for SQLite)
-      for (const chunk of _.chunk(tree, 100)) {
+      if ((WIKI.config.db.type !== 'sqlite'))
-        await WIKI.models.knex.table('pageTree').insert(chunk)
+        for (const chunk of _.chunk(tree, 100)) {
+          await WIKI.models.knex.table('pageTree').insert(chunk)
+        }
+      } else {
+        for (const chunk of _.chunk(tree, 60)) {
+          await WIKI.models.knex.table('pageTree').insert(chunk)
+        }
      }
    }