fix: serialization errors (#414)

WalletConnect · Mar 13, 2024 · 40bdb01 · 40bdb01
1 parent e27183b
commit 40bdb01
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/model/helpers.rs b/src/model/helpers.rs
@@ -732,9 +732,11 @@ pub async fn upsert_subscription_watcher(
     let start = Instant::now();
     let mut txn = postgres.begin().await?;
     // https://stackoverflow.com/a/48730873
-    sqlx::query::<Postgres>("SET TRANSACTION ISOLATION LEVEL SERIALIZABLE") // TODO serialization errors not handled
-        .execute(&mut *txn)
-        .await?;
+    // Allow phantom reads; going above the watcher limit is not a big deal and handling
+    // serialization errors is not worth the effort
+    // sqlx::query::<Postgres>("SET TRANSACTION ISOLATION LEVEL SERIALIZABLE")
+    //     .execute(&mut *txn)
+    //     .await?;
     let result = sqlx::query_as::<Postgres, ()>(query)
         .bind(account.as_ref())
         .bind(project)