Send target to backend before local processing (#5980)

* Send target to backend before local processing * Create lemon-ravens-sneeze.md
firebase · Feb 9, 2022 · e961968 · e961968
1 parent bf93a38
commit e961968
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.changeset/lemon-ravens-sneeze.md b/.changeset/lemon-ravens-sneeze.md
@@ -0,0 +1,5 @@
+---
+"@firebase/firestore": patch
+---
+
+Queries are now send to the backend before the SDK starts local processing, which reduces overall Query latency.
diff --git a/packages/firestore/src/core/sync_engine_impl.ts b/packages/firestore/src/core/sync_engine_impl.ts
@@ -314,6 +314,9 @@ export async function syncEngineListen(
       syncEngineImpl.localStore,
       queryToTarget(query)
     );
+    if (syncEngineImpl.isPrimaryClient) {
+      remoteStoreListen(syncEngineImpl.remoteStore, targetData);
+    }
 
     const status = syncEngineImpl.sharedClientState.addLocalQueryTarget(
       targetData.targetId
@@ -325,9 +328,6 @@ export async function syncEngineListen(
       targetId,
       status === 'current'
     );
-    if (syncEngineImpl.isPrimaryClient) {
-      remoteStoreListen(syncEngineImpl.remoteStore, targetData);
-    }
   }
 
   return viewSnapshot;