Browse Source

fix: max-sql-params

Tienson Qin 1 month ago
parent
commit
573c915cf0

+ 1 - 1
deps/db-sync/src/logseq/db_sync/batch.cljs

@@ -1,7 +1,7 @@
 (ns logseq.db-sync.batch
   (:require [clojure.string :as string]))
 
-(def ^:private max-sql-params 999)
+(def ^:private max-sql-params 99)
 (def ^:private row-param-count 3)
 
 (defn rows->insert-batches

+ 5 - 2
deps/db-sync/src/logseq/db_sync/worker.cljs

@@ -255,8 +255,10 @@
 (defn- import-snapshot-rows!
   [sql table rows]
   (when (seq rows)
-    (doseq [{:keys [sql args]} (batch/rows->insert-batches table rows nil)]
-      (apply common/sql-exec sql sql args))))
+    (doseq [batch (batch/rows->insert-batches table rows nil)]
+      (let [sql-str (:sql batch)
+            args (:args batch)]
+        (apply common/sql-exec sql sql-str args)))))
 
 (defn- finalize-import!
   [^js self reset?]
@@ -592,6 +594,7 @@
 ;;      :datoms (common/write-transit datoms)}))
 
 (defn- import-snapshot! [^js self rows _reset?]
+  (prn :debug :rows-count (count rows))
   (let [sql (.-sql self)]
     (ensure-schema! self)
     (import-snapshot-rows! sql "kvs_import" rows)))

+ 0 - 1
deps/db-sync/worker/wrangler.toml

@@ -38,7 +38,6 @@ name = "logseq-sync"
 COGNITO_JWKS_URL = "https://cognito-idp.us-east-1.amazonaws.com/us-east-1_dtagLnju8/.well-known/jwks.json"
 COGNITO_ISSUER = "https://cognito-idp.us-east-1.amazonaws.com/us-east-1_dtagLnju8"
 COGNITO_CLIENT_ID = "69cs1lgme7p8kbgld8n5kseii6"
-ENABLE_DB_SYNC_LOCAL = "true"
 
 [[env.staging.durable_objects.bindings]]
 name = "LOGSEQ_SYNC_DO"