Fixed the bug (#23)

- Syntax error occurred when SQL contains partition queries.

Author: hatajoe

builder/builder.go

@@ -60,7 +60,7 @@ func willAlterTableCharacterSet(old, new *mysql.Table) string {
 		alter = append(alter, new.ToConvertCharsetSQL())
 		old.TableCollation = new.TableCollation
 	}
-	return new.ToAlterSQL(alter)
+	return new.ToAlterSQL(alter, "")
 }
 
 func willAlterColumnCharacterSet(old, new *mysql.Table) []string {
@@ -81,7 +81,7 @@ func willAlterColumnCharacterSet(old, new *mysql.Table) []string {
 			continue
 		}
 		oldCols[colName].CollationName = newCol.CollationName
-		sqls = append(sqls, new.ToAlterSQL([]string{newCol.ToModifyCharsetSQL()}))
+		sqls = append(sqls, new.ToAlterSQL([]string{newCol.ToModifyCharsetSQL()}, ""))
 	}
 	return sqls
 }
@@ -100,11 +100,7 @@ func willAlter(old, new *mysql.Table) string {
 	alter = append(alter, willAddColumn(old, new)...)
 	alter = append(alter, willAddIndex(old, new)...)
 	alter = append(alter, willModifyColumn(old, new)...)
-	sql := willModifyPartition(old, new)
-	if sql != "" {
-		alter = append(alter, sql)
-	}
-	return new.ToAlterSQL(alter)
+	return new.ToAlterSQL(alter, willModifyPartition(old, new))
 }
 
 func willAddColumn(old, new *mysql.Table) []string {

dialect/mysql/table.go

@@ -39,7 +39,7 @@ var (
 	%s
 ) engine=%s default charset=%s %s`
 	dropSQLFmt  string = `drop table if exists %s`
-	alterSQLFmt string = `alter table %s
+	alterSQLFmt string = `alter table %s %s
 	%s`
 )
 
@@ -56,11 +56,11 @@ func (m *Table) GetCharset() string {
 	return seg[0]
 }
 
-func (m *Table) ToAlterSQL(sqls []string) string {
+func (m *Table) ToAlterSQL(sqls []string, partitionSql string) string {
 	if len(sqls) <= 0 {
 		return ""
 	}
-	return fmt.Sprintf(alterSQLFmt, m.GetFormatedTableName(), strings.Join(sqls, ",\n	"))
+	return fmt.Sprintf(alterSQLFmt, m.GetFormatedTableName(), strings.Join(sqls, ",\n	"), partitionSql)
 }
 
 func (m Tables) GetFormatedTableNames() []string {