Move createTime to the LAST index key component, but did not fix unit tests

Author: dconeybe

packages/firestore/src/local/indexeddb_remote_document_cache.ts

@@ -289,19 +289,19 @@ class IndexedDbRemoteDocumentCacheImpl implements IndexedDbRemoteDocumentCache {
   ): PersistencePromise<MutableDocumentMap> {
     const collection = query.path;
     const startKey: DbRemoteDocumentCollectionIndexKey = [
-      MIN_CREATE_TIME,
       collection.popLast().toArray(),
       collection.lastSegment(),
       toDbTimestampKey(offset.readTime),
       offset.documentKey.path.isEmpty()
         ? ''
-        : offset.documentKey.path.lastSegment()
+        : offset.documentKey.path.lastSegment(),
+      ''
     ];
     const endKey: DbRemoteDocumentCollectionIndexKey = [
-      MAX_CREATE_TIME,
       collection.popLast().toArray(),
       collection.lastSegment(),
       [Number.MAX_SAFE_INTEGER, Number.MAX_SAFE_INTEGER],
+      '',
       ''
     ];
 

packages/firestore/src/local/indexeddb_sentinels.ts

@@ -199,11 +199,11 @@ export const DbRemoteDocumentCollectionGroupIndexPath = [
 export const DbRemoteDocumentCollectionIndex = 'collectionIndex';
 
 export const DbRemoteDocumentCollectionIndexPath = [
-  'document.createTime',
   'prefixPath',
   'collectionGroup',
   'readTime',
-  'documentId'
+  'documentId',
+  'document.createTime'
 ];
 
 export const MIN_CREATE_TIME = '' as const;
@@ -218,11 +218,11 @@ export const MAX_CREATE_TIME = '\uFFFF' as const;
  * filtering.
  */
 export type DbRemoteDocumentCollectionIndexKey = [
-  /** document.createTime */ typeof MIN_CREATE_TIME | typeof MAX_CREATE_TIME,
   /** path to collection */ string[],
   /** collection group */ string,
   /** read time */ DbTimestampKey,
-  /** document ID */ string
+  /** document ID */ string,
+  /** document.createTime */ typeof MIN_CREATE_TIME | typeof MAX_CREATE_TIME
 ];
 
 export const DbRemoteDocumentGlobalStore = 'remoteDocumentGlobal';