octoleo/gh-ost
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #79 from github/noop-drop-ghost

Browse Source 
on noop operation, drop ghost table at end
This commit is contained in:
Shlomi Noach 2016-06-22 10:49:21 +02:00 committed by GitHub
commit 45625a6a6d
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
go/logic/migrator.go
View File

@ -1177,10 +1177,12 @@ func (this *Migrator) finalCleanup() error {
return err
}
if this.migrationContext.OkToDropTable && !this.migrationContext.TestOnReplica {
dropTableFunc := func() error {
return this.applier.dropTable(this.migrationContext.GetOldTableName())
if err := this.retryOperation(this.applier.DropOldTable); err != nil {
return err
}
if err := this.retryOperation(dropTableFunc); err != nil {
}
if this.migrationContext.Noop {
if err := this.retryOperation(this.applier.DropGhostTable); err != nil {
return err
}
}