Fixes apply full schema snapshot (#100)

microsoft · Sep 25, 2020 · 7573aba · 7573aba
1 parent cd31949
commit 7573aba
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/tools/SchemaManager/Commands/ApplyCommand.cs b/tools/SchemaManager/Commands/ApplyCommand.cs
@@ -85,7 +85,8 @@ await Policy.Handle<SchemaManagerException>()
                 {
                     await ValidateVersionCompatibility(schemaClient, availableVersions.Last().Id);
                 }
-                else if (availableVersions.First().Id == 1)
+
+                if (availableVersions.First().Id == 1)
                 {
                     // Upgrade schema directly to the latest schema version
                     Console.WriteLine(string.Format(Resources.SchemaMigrationStartedMessage, availableVersions.Last().Id));

diff --git a/tools/SchemaManager/SchemaDataStore.cs b/tools/SchemaManager/SchemaDataStore.cs
@@ -30,11 +30,7 @@ public static void ExecuteScriptAndCompleteSchemaVersion(string connectionString
 
                     serverConnection.BeginTransaction();
 
-                    // Since version 3 is just the base schema which must already be executed at the start of apply command and can be skipped
-                    if (version != 3)
-                    {
-                        server.ConnectionContext.ExecuteNonQuery(script);
-                    }
+                    server.ConnectionContext.ExecuteNonQuery(script);
 
                     UpsertSchemaVersion(connection, version, Completed);