[release-19.0] fix: don't forget DISTINCT for derived tables (#15672) (…

…#15678)

Co-authored-by: vitess-bot[bot] <108069721+vitess-bot[bot]@users.noreply.github.com>

go/vt/vtgate/planbuilder/operators/SQL_builder.go

@@ -612,6 +612,7 @@ func buildDerivedSelect(op *Horizon, qb *queryBuilder, sel *sqlparser.Select) {
 	sel.GroupBy = opQuery.GroupBy
 	sel.Having = mergeHaving(sel.Having, opQuery.Having)
 	sel.SelectExprs = opQuery.SelectExprs
+	sel.Distinct = opQuery.Distinct
 	qb.addTableExpr(op.Alias, op.Alias, TableID(op), &sqlparser.DerivedTable{
 		Select: sel,
 	}, nil, op.ColumnAliases)

go/vt/vtgate/planbuilder/testdata/from_cases.json

@@ -409,6 +409,28 @@
       ]
     }
   },
+  {
+    "comment": "DISTINCT inside derived table",
+    "query": "select * from (select distinct name from user) as t",
+    "plan": {
+      "QueryType": "SELECT",
+      "Original": "select * from (select distinct name from user) as t",
+      "Instructions": {
+        "OperatorType": "Route",
+        "Variant": "Scatter",
+        "Keyspace": {
+          "Name": "user",
+          "Sharded": true
+        },
+        "FieldQuery": "select `name` from (select `name` from `user` where 1 != 1) as t where 1 != 1",
+        "Query": "select `name` from (select distinct `name` from `user`) as t",
+        "Table": "`user`"
+      },
+      "TablesUsed": [
+        "user.user"
+      ]
+    }
+  },
   {
     "comment": "',' join unsharded",
     "query": "select u1.a, u2.a from unsharded u1, unsharded u2",