handle merged routes correctly

Signed-off-by: Andres Taylor <andres@planetscale.com>
vitessio · Sep 16, 2024 · 9d29c98 · 9d29c98
1 parent 5a5664c
commit 9d29c98
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/go/vt/vtgate/planbuilder/operators/route.go b/go/vt/vtgate/planbuilder/operators/route.go
@@ -719,7 +719,8 @@ func (r *Route) GetOrdering(ctx *plancontext.PlanningContext) []OrderBy {
 // in Inputs() and thus not a part of the operator tree
 func (r *Route) TablesUsed(in []string) []string {
 	for _, mw := range r.MergedWith {
-		in = append(in, mw.TablesUsed(in)...)
+		used := TablesUsed(mw)
+		in = append(in, used...)
 	}
 	return in
 }

diff --git a/go/vt/vtgate/planbuilder/testdata/onecase.json b/go/vt/vtgate/planbuilder/testdata/onecase.json
@@ -4,5 +4,4 @@
     "query": "",
     "plan": {
     }
-  }
-]
+  }