NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf583633

committed

Fix breakage in AlterFunction().

An ALTER FUNCTION command that tried to update both the function'sproparallel property and its proconfig list failed to do the former,because it stored the new proparallel value into a tuple that wasno longer the interesting one. Carelessness in7aea8e4.(I did not bother with a regression test, because the only likelyfuture breakage would be for someone to ignore the comment I addedand add some other field update after the heap_modify_tuple step.A test using existing function properties could not catch that.)Per report from Bryn Llewellyn. Back-patch to all supported branches.Discussion:https://postgr.es/m/8AC9A37F-99BD-446F-A2F7-B89AD0022774@yugabyte.com

1 parent82d4a17 commitf583633Copy full SHA for f583633

File tree

1 file changed

-2

lines changed

src/backend/commands
- functioncmds.c

1 file changed

-2

lines changed

`‎src/backend/commands/functioncmds.c`

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1354,6 +1354,8 @@ AlterFunction(ParseState pstate, AlterFunctionStmt stmt)`
`1354`	`1354`
`1355`	`1355`	`procForm->prosupport=newsupport;`
`1356`	`1356`	`}`
	`1357`	`+if (parallel_item)`
	`1358`	`+procForm->proparallel=interpret_func_parallel(parallel_item);`
`1357`	`1359`	`if (set_items)`
`1358`	`1360`	`{`
`1359`	`1361`	`Datumdatum;`
`@@ -1388,8 +1390,7 @@ AlterFunction(ParseState pstate, AlterFunctionStmt stmt)`
`1388`	`1390`	`tup=heap_modify_tuple(tup,RelationGetDescr(rel),`
`1389`	`1391`	`repl_val,repl_null,repl_repl);`
`1390`	`1392`	`}`
`1391`		`-if (parallel_item)`
`1392`		`-procForm->proparallel=interpret_func_parallel(parallel_item);`
	`1393`	`+/* DO NOT put more touches of procForm below here; it's now dangling. */`
`1393`	`1394`
`1394`	`1395`	`/* Do the update */`
`1395`	`1396`	`CatalogTupleUpdate(rel,&tup->t_self,tup);`

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!

Commitf583633

File tree

1 file changed

1 file changed

`‎src/backend/commands/functioncmds.c`

0 commit comments