You've already forked go-clickhouse
mirror of
https://github.com/uptrace/go-clickhouse.git
synced 2025-06-29 00:21:46 +02:00
fix: change rollback to always record migrations
This commit is contained in:
@ -186,15 +186,16 @@ func (m *Migrator) Rollback(ctx context.Context, opts ...MigrationOption) (*Migr
|
|||||||
for i := len(lastGroup.Migrations) - 1; i >= 0; i-- {
|
for i := len(lastGroup.Migrations) - 1; i >= 0; i-- {
|
||||||
migration := &lastGroup.Migrations[i]
|
migration := &lastGroup.Migrations[i]
|
||||||
|
|
||||||
|
// Always mark migration as unapplied to match migrate behavior.
|
||||||
|
if err := m.MarkUnapplied(ctx, migration); err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
|
||||||
if !cfg.nop && migration.Down != nil {
|
if !cfg.nop && migration.Down != nil {
|
||||||
if err := migration.Down(ctx, m.db); err != nil {
|
if err := migration.Down(ctx, m.db); err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if err := m.MarkUnapplied(ctx, migration); err != nil {
|
|
||||||
return nil, err
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return lastGroup, nil
|
return lastGroup, nil
|
||||||
|
Reference in New Issue
Block a user