Skip to content

Commit 0fbe50f

Browse files
authored
Backport 12634 (vitessio#12680)
Signed-off-by: Florent Poinsard <florent.poinsard@outlook.fr>
1 parent d8c5160 commit 0fbe50f

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

go/vt/vtgate/planbuilder/physical/operator_funcs.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,11 +231,10 @@ func PushOutputColumns(ctx *plancontext.PlanningContext, op abstract.PhysicalOpe
231231
var pos int
232232
op.ColumnsOffset, pos = addToIntSlice(op.ColumnsOffset, i)
233233
offsets = append(offsets, pos)
234-
// skip adding to columns as it exists already.
234+
op.Columns = append(op.Columns, col)
235235
if i > -1 {
236236
continue
237237
}
238-
op.Columns = append(op.Columns, col)
239238
noQualifierNames = append(noQualifierNames, sqlparser.NewColName(col.Name.String()))
240239
}
241240
if len(noQualifierNames) > 0 {

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1287,7 +1287,7 @@
12871287
"Instructions": {
12881288
"OperatorType": "Join",
12891289
"Variant": "Join",
1290-
"JoinColumnIndexes": "L:0",
1290+
"JoinColumnIndexes": "L:1",
12911291
"JoinVars": {
12921292
"x_COLUMN_NAME": 0
12931293
},
@@ -1300,8 +1300,8 @@
13001300
"Name": "main",
13011301
"Sharded": false
13021302
},
1303-
"FieldQuery": "select x.table_name from (select a.CONSTRAINT_CATALOG, a.CONSTRAINT_SCHEMA, a.CONSTRAINT_NAME, a.TABLE_CATALOG, a.TABLE_SCHEMA, a.TABLE_NAME, a.COLUMN_NAME, a.ORDINAL_POSITION, a.POSITION_IN_UNIQUE_CONSTRAINT, a.REFERENCED_TABLE_SCHEMA, a.REFERENCED_TABLE_NAME, a.REFERENCED_COLUMN_NAME from information_schema.key_column_usage as a where 1 != 1) as x where 1 != 1",
1304-
"Query": "select x.table_name from (select a.CONSTRAINT_CATALOG, a.CONSTRAINT_SCHEMA, a.CONSTRAINT_NAME, a.TABLE_CATALOG, a.TABLE_SCHEMA, a.TABLE_NAME, a.COLUMN_NAME, a.ORDINAL_POSITION, a.POSITION_IN_UNIQUE_CONSTRAINT, a.REFERENCED_TABLE_SCHEMA, a.REFERENCED_TABLE_NAME, a.REFERENCED_COLUMN_NAME from information_schema.key_column_usage as a) as x",
1303+
"FieldQuery": "select x.COLUMN_NAME, x.table_name from (select a.CONSTRAINT_CATALOG, a.CONSTRAINT_SCHEMA, a.CONSTRAINT_NAME, a.TABLE_CATALOG, a.TABLE_SCHEMA, a.TABLE_NAME, a.COLUMN_NAME, a.ORDINAL_POSITION, a.POSITION_IN_UNIQUE_CONSTRAINT, a.REFERENCED_TABLE_SCHEMA, a.REFERENCED_TABLE_NAME, a.REFERENCED_COLUMN_NAME from information_schema.key_column_usage as a where 1 != 1) as x where 1 != 1",
1304+
"Query": "select x.COLUMN_NAME, x.table_name from (select a.CONSTRAINT_CATALOG, a.CONSTRAINT_SCHEMA, a.CONSTRAINT_NAME, a.TABLE_CATALOG, a.TABLE_SCHEMA, a.TABLE_NAME, a.COLUMN_NAME, a.ORDINAL_POSITION, a.POSITION_IN_UNIQUE_CONSTRAINT, a.REFERENCED_TABLE_SCHEMA, a.REFERENCED_TABLE_NAME, a.REFERENCED_COLUMN_NAME from information_schema.key_column_usage as a) as x",
13051305
"Table": "information_schema.key_column_usage"
13061306
},
13071307
{

0 commit comments

Comments
 (0)