Performance - only check for non-transactional tables when updating schema

1 files changed, 8 insertions, 6 deletions

diff --git a/app/Services/MigrationService.php b/app/Services/MigrationService.php
index 5e97f6fded..46e5ecab5e 100644
--- a/app/Services/MigrationService.php
+++ b/app/Services/MigrationService.php
@@ -47,18 +47,20 @@ class MigrationService
     public function updateSchema($namespace, $schema_name, $target_version): bool
     {
         try {
-            $this->transactionalTables();
-        } catch (PDOException $ex) {
-            // There is probably nothing we can do.
-        }
-
-        try {
             $current_version = (int) Site::getPreference($schema_name);
         } catch (PDOException $ex) {
             // During initial installation, the site_preference table won’t exist.
             $current_version = 0;
         }
 
+        if ($current_version < $target_version) {
+            try {
+                $this->transactionalTables();
+            } catch (PDOException $ex) {
+                // There is probably nothing we can do.
+            }
+        }
+
         $updates_applied = false;
 
         // Update the schema, one version at a time.