NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit21daada

committed

Dept of second thoughts: keep aliasp_item in sync with tlistitem.

Commitd5b760e wasn't quite right, on second thought: if thecaller didn't ask for column names then it would happily emitmore Vars than if the caller did ask for column names. Thisis surely not a good idea. Advance the aliasp_item whether ornot we're preparing a colnames list.

1 parent7e5e8b3 commit21daadaCopy full SHA for 21daada

File tree

1 file changed

-1

lines changed

src/backend/parser
- parse_relation.c

1 file changed

-1

lines changed

`‎src/backend/parser/parse_relation.c`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -2028,7 +2028,6 @@ expandRTE(RangeTblEntry *rte, int rtindex, int sublevels_up,`
`2028`	`2028`	`char*label=strVal(lfirst(aliasp_item));`
`2029`	`2029`
`2030`	`2030`	`colnames=lappend(colnames,makeString(pstrdup(label)));`
`2031`		`-aliasp_item=lnext(aliasp_item);`
`2032`	`2031`	`}`
`2033`	`2032`
`2034`	`2033`	`if (colvars)`
`@@ -2044,6 +2043,8 @@ expandRTE(RangeTblEntry *rte, int rtindex, int sublevels_up,`
`2044`	`2043`
`2045`	`2044`	`colvars=lappend(colvars,varnode);`
`2046`	`2045`	`}`
	`2046`	`+`
	`2047`	`+aliasp_item=lnext(aliasp_item);`
`2047`	`2048`	`}`
`2048`	`2049`	`}`
`2049`	`2050`	`break;`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit21daada

File tree

1 file changed

1 file changed

`‎src/backend/parser/parse_relation.c`

0 commit comments