fix: onSuccess hook to run after INSTANT (#1463)

previously, no hooks would run after a successful INSTANT DDL, which is an unexpected behavior.
github · Oct 31, 2024 · a6ccd3f · a6ccd3f
1 parent 7c30fb0
commit a6ccd3f
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/go/logic/migrator.go b/go/logic/migrator.go
@@ -367,6 +367,9 @@ func (this *Migrator) Migrate() (err error) {
 		} else {
 			this.migrationContext.Log.Infof("Attempting to execute alter with ALGORITHM=INSTANT")
 			if err := this.applier.AttemptInstantDDL(); err == nil {
+				if err := this.hooksExecutor.onSuccess(); err != nil {
+					return err
+				}
 				this.migrationContext.Log.Infof("Success! table %s.%s migrated instantly", sql.EscapeName(this.migrationContext.DatabaseName), sql.EscapeName(this.migrationContext.OriginalTableName))
 				return nil
 			} else {