fixup! Better support for merging UNION routes that use IN conditions.

arthurschreiber · arthurschreiber · commit fb9e8ef675a2 · 2025-05-22T18:19:22.000Z
Signed-off-by: Arthur Schreiber &lt;arthurschreiber@github.com&gt;
diff --git a/go/vt/vtgate/planbuilder/operators/route_planning.go b/go/vt/vtgate/planbuilder/operators/route_planning.go
@@ -525,7 +525,7 @@ func gen4ValEqual(ctx *plancontext.PlanningContext, a, b sqlparser.Expr) (bool,
 		}
 
 		return aVal.Type == bVal.Type && bytes.Equal(aVal.Value, bVal.Value),
-			[]engine.Condition{engine.Condition{A: a.Name, B: b.Name}}
+			[]engine.Condition{{A: a.Name, B: b.Name}}
 
 	case *sqlparser.Literal:
 		b, ok := b.(*sqlparser.Literal)

Original file line number	Diff line number	Diff line change
`@@ -525,7 +525,7 @@ func gen4ValEqual(ctx *plancontext.PlanningContext, a, b sqlparser.Expr) (bool,`
`525`	`525`	`}`
`526`	`526`
`527`	`527`	`return aVal.Type == bVal.Type && bytes.Equal(aVal.Value, bVal.Value),`
`528`		`- []engine.Condition{engine.Condition{A: a.Name, B: b.Name}}`
	`528`	`+ []engine.Condition{{A: a.Name, B: b.Name}}`
`529`	`529`
`530`	`530`	`case *sqlparser.Literal:`
`531`	`531`	`b, ok := b.(*sqlparser.Literal)`