fix and update ddl test with plan type

Signed-off-by: Harshit Gangal <harshit@planetscale.com>
planetscale · Feb 10, 2025 · cfc2806 · cfc2806
1 parent d2062ea
commit cfc2806
Show file tree

Hide file tree

Showing 5 changed files with 1,618 additions and 0 deletions.
diff --git a/go/vt/vtgate/engine/ddl.go b/go/vt/vtgate/engine/ddl.go
@@ -81,6 +81,9 @@ func (ddl *DDL) GetTableName() string {
 func (ddl *DDL) isOnlineSchemaDDL() bool {
 	switch ddl.DDL.GetAction() {
 	case sqlparser.CreateDDLAction, sqlparser.DropDDLAction, sqlparser.AlterDDLAction:
+		if ddl.OnlineDDL == nil || ddl.OnlineDDL.DDLStrategySetting == nil {
+			return false
+		}
 		return !ddl.OnlineDDL.DDLStrategySetting.Strategy.IsDirect()
 	}
 	return false