remove onecase

Signed-off-by: Harshit Gangal <harshit@planetscale.com>
vitessio · Jun 10, 2024 · 0107ddc · 0107ddc
1 parent 67d5523
commit 0107ddc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/go/vt/vtgate/planbuilder/operators/subquery_planning.go b/go/vt/vtgate/planbuilder/operators/subquery_planning.go
@@ -157,7 +157,7 @@ func rewriteMergedSubqueryExpr(ctx *plancontext.PlanningContext, se SubQueryExpr
 							return true
 						}
 					case *sqlparser.Argument:
-						if expr.Name != sq. /**/ ArgName {
+						if expr.Name != sq.ArgName {
 							return true
 						}
 					default:

diff --git a/go/vt/vtgate/planbuilder/testdata/onecase.json b/go/vt/vtgate/planbuilder/testdata/onecase.json
@@ -1,8 +1,9 @@
 [
   {
     "comment": "Add your test case here for debugging and run go test -run=One.",
-    "query": "SELECT id, name, (SELECT COUNT(*) FROM user_extra) AS total_extra_count FROM user GROUP BY id, name ORDER BY total_extra_count",
+    "query": "",
     "plan": {
+
     }
   }
 ]