fix: rebuild-tree exceeds mssql parameter limit (#1328)

* fix: rebuild-tree exceeds mssql's parameter limit * docs: details per query max parameters limits Co-authored-by: Nicolas Giard <github@ngpixel.com>

fix: rebuild-tree exceeds mssql parameter limit (#1328)
a257831c · alancnet · Nicolas Giard · a457e82b · a257831c
Commit a257831c authored Dec 19, 2019 by alancnet Committed by Nicolas Giard Dec 19, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

rebuild-tree.js server/jobs/rebuild-tree.js +4 -1

No files found.
--- a/server/jobs/rebuild-tree.js
+++ b/server/jobs/rebuild-tree.js
@@ -53,7 +53,10 @@ module.exports = async (pageId) => {

    await WIKI.models.knex.table('pageTree').truncate()
    if (tree.length > 0) {
-      await WIKI.models.knex.table('pageTree').insert(tree)
+      // -> Save in chunks, because of per query max parameters (35k Postgres, 2k MSSQL, 1k for SQLite)
+      for (const chunk of _.chunk(tree, 100)) {
+        await WIKI.models.knex.table('pageTree').insert(chunk)
+      }
    }

    await WIKI.models.knex.destroy()