Fill read_time

Author: schmidt-sebastian

firebase-firestore/src/main/java/com/google/firebase/firestore/local/SQLiteSchema.java

@@ -49,7 +49,7 @@ class SQLiteSchema {
    * The version of the schema. Increase this by one for each migration added to runMigrations
    * below.
    */
-  static final int VERSION = 13;
+  static final int VERSION = 14;
 
   static final int OVERLAY_SUPPORT_VERSION = VERSION + 1;
 
@@ -178,6 +178,10 @@ void runSchemaUpgrades(int fromVersion, int toVersion) {
       ensurePathLength();
     }
 
+    if (fromVersion < 14 && toVersion >= 14) {
+      ensureReadTime();
+    }
+
     /*
      * Adding a new migration? READ THIS FIRST!
      *
@@ -658,6 +662,12 @@ private void ensurePathLength() {
     } while (resultsRemaining[0]);
   }
 
+  /** Initialize the remote_document's read_time column with 0 values if they are not set. */
+  private void ensureReadTime() {
+    db.execSQL(
+        "UPDATE remote_documents SET read_time_seconds = 0, read_time_nanos = 0 WHERE read_time_seconds IS NULL");
+  }
+
   private void createBundleCache() {
     ifTablesDontExist(
         new String[] {"bundles", "named_queries"},

firebase-firestore/src/test/java/com/google/firebase/firestore/local/SQLiteSchemaTest.java

@@ -426,7 +426,7 @@ public void existingDocumentsRemainReadableAfterIndexFreeMigration() {
         new Object[] {encode(path("coll/existing")), createDummyDocument("coll/existing")});
 
     // Run the index-free migration.
-    schema.runSchemaUpgrades(8, 10);
+    schema.runSchemaUpgrades(8, 14);
     db.execSQL(
         "INSERT INTO remote_documents (path, read_time_seconds, read_time_nanos, contents) VALUES (?, ?, ?, ?)",
         new Object[] {encode(path("coll/old")), 0, 1000, createDummyDocument("coll/old")});