Commitbe87143

committed

Fix bogus range_table_mutator() logic for RangeTblEntry.tablesample.

Must make a copy of the TableSampleClause node; the previous codingmodified the input data structure in-place.Petr Jelinek

1 parented16f73 commitbe87143Copy full SHA for be87143

File tree

-3

lines changed

-3

lines changed

Lines changed: 7 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -2870,10 +2870,14 @@ range_table_mutator(List *rtable,`
`2870`	`2870`	`caseRTE_RELATION:`
`2871`	`2871`	`if (rte->tablesample)`
`2872`	`2872`	`{`
`2873`		`-MUTATE(rte->tablesample->args,rte->tablesample->args,`
	`2873`	`+CHECKFLATCOPY(newrte->tablesample,rte->tablesample,`
	`2874`	`+TableSampleClause);`
	`2875`	`+MUTATE(newrte->tablesample->args,`
	`2876`	`+newrte->tablesample->args,`
`2874`	`2877`	`List*);`
`2875`		`-MUTATE(rte->tablesample->repeatable,`
`2876`		`-rte->tablesample->repeatable,Node*);`
	`2878`	`+MUTATE(newrte->tablesample->repeatable,`
	`2879`	`+newrte->tablesample->repeatable,`
	`2880`	`+Node*);`
`2877`	`2881`	`}`
`2878`	`2882`	`break;`
`2879`	`2883`	`caseRTE_CTE:`

Comments

(0)