Index-Free: Track readTime in the RemoteDocument store (#2125)

Author: schmidt-sebastian

packages/firestore/src/local/indexeddb_schema.ts

@@ -138,10 +138,13 @@ export class SchemaConverter implements SimpleDbSchemaConverter {
 
     if (fromVersion < 9 && toVersion >= 9) {
       p = p.next(() => {
+<<<<<<< HEAD
         // Multi-Tab used to manage its own changelog, but this has been moved
         // to the DbRemoteDocument object store itself. Since the previous change
         // log only contained transient data, we can drop its object store.
         dropRemoteDocumentChangesStore(db);
+=======
+>>>>>>> b4d70ace... Index-Free: Track readTime in the RemoteDocument store (#2125)
         createRemoteDocumentReadTimeIndex(txn);
       });
     }
@@ -610,6 +613,7 @@ export class DbRemoteDocument {
   static store = 'remoteDocuments';
 
   /**
+<<<<<<< HEAD
    * An index that provides access to all entries sorted by read time (which
    * corresponds to the last modification time of each row).
    *
@@ -620,6 +624,8 @@ export class DbRemoteDocument {
   static readTimeIndexPath = 'readTime';
 
   /**
+=======
+>>>>>>> b4d70ace... Index-Free: Track readTime in the RemoteDocument store (#2125)
    * An index that provides access to documents in a collection sorted by read
    * time.
    *
@@ -997,6 +1003,19 @@ function createRemoteDocumentReadTimeIndex(txn: IDBTransaction): void {
   );
 }
 
+/**
+ * Creates indices on the RemoteDocuments store used for both multi-tab
+ * and Index-Free queries.
+ */
+function createRemoteDocumentReadTimeIndex(txn: IDBTransaction): void {
+  const remoteDocumentStore = txn.objectStore(DbRemoteDocument.store);
+  remoteDocumentStore.createIndex(
+    DbRemoteDocument.collectionReadTimeIndex,
+    DbRemoteDocument.collectionReadTimeIndexPath,
+    { unique: false }
+  );
+}
+
 /**
  * A record of the metadata state of each client.
  *

packages/firestore/src/local/local_serializer.ts

@@ -123,11 +123,14 @@ export class LocalSerializer {
     return [timestamp.seconds, timestamp.nanoseconds];
   }
 
+<<<<<<< HEAD
   fromDbTimestampKey(dbTimestampKey: DbTimestampKey): SnapshotVersion {
     const timestamp = new Timestamp(dbTimestampKey[0], dbTimestampKey[1]);
     return SnapshotVersion.fromTimestamp(timestamp);
   }
 
+=======
+>>>>>>> b4d70ace... Index-Free: Track readTime in the RemoteDocument store (#2125)
   private toDbTimestamp(snapshotVersion: SnapshotVersion): DbTimestamp {
     const timestamp = snapshotVersion.toTimestamp();
     return new DbTimestamp(timestamp.seconds, timestamp.nanoseconds);

packages/firestore/src/local/local_store.ts

@@ -450,9 +450,13 @@ export class LocalStore {
    * queue.
    */
   applyRemoteEvent(remoteEvent: RemoteEvent): Promise<MaybeDocumentMap> {
+<<<<<<< HEAD
     const documentBuffer = this.remoteDocuments.newChangeBuffer({
       createSentinelDocumentsToTrackDeletes: true
     });
+=======
+    const documentBuffer = this.remoteDocuments.newChangeBuffer();
+>>>>>>> b4d70ace... Index-Free: Track readTime in the RemoteDocument store (#2125)
     const remoteVersion = remoteEvent.snapshotVersion;
     return this.persistence.runTransaction(
       'Apply remote event',