NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commite06b9e9

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 parent9d15b8b commite06b9e9Copy full SHA for e06b9e9

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
`@@ -1725,7 +1725,6 @@ expandRTE(RangeTblEntry *rte, int rtindex, int sublevels_up,`
`1725`	`1725`	`char*label=strVal(lfirst(aliasp_item));`
`1726`	`1726`
`1727`	`1727`	`colnames=lappend(colnames,makeString(pstrdup(label)));`
`1728`		`-aliasp_item=lnext(aliasp_item);`
`1729`	`1728`	`}`
`1730`	`1729`
`1731`	`1730`	`if (colvars)`
`@@ -1741,6 +1740,8 @@ expandRTE(RangeTblEntry *rte, int rtindex, int sublevels_up,`
`1741`	`1740`
`1742`	`1741`	`colvars=lappend(colvars,varnode);`
`1743`	`1742`	`}`
	`1743`	`+`
	`1744`	`+aliasp_item=lnext(aliasp_item);`
`1744`	`1745`	`}`
`1745`	`1746`	`}`
`1746`	`1747`	`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!

Commite06b9e9

File tree

1 file changed

1 file changed

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

0 commit comments