Fix breakage in AlterFunction().

tglsfdc · tglsfdc · commit 481a99811a41 · 2022-04-19T23:03:59.000-04:00
An ALTER FUNCTION command that tried to update both the function's proparallel property and its proconfig list failed to do the former, because it stored the new proparallel value into a tuple that was no longer the interesting one. Carelessness in 7aea8e4. (I did not bother with a regression test, because the only likely future breakage would be for someone to ignore the comment I added and 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
diff --git a/src/backend/commands/functioncmds.c b/src/backend/commands/functioncmds.c
@@ -1352,6 +1352,8 @@ AlterFunction(ParseState *pstate, AlterFunctionStmt *stmt)
 
 		procForm->prosupport = newsupport;
 	}
+	if (parallel_item)
+		procForm->proparallel = interpret_func_parallel(parallel_item);
 	if (set_items)
 	{
 		Datum		datum;
@@ -1386,8 +1388,7 @@ AlterFunction(ParseState *pstate, AlterFunctionStmt *stmt)
 		tup = heap_modify_tuple(tup, RelationGetDescr(rel),
 								repl_val, repl_null, repl_repl);
 	}
-	if (parallel_item)
-		procForm->proparallel = interpret_func_parallel(parallel_item);
+	/* DO NOT put more touches of procForm below here; it's now dangling. */
 
 	/* Do the update */
 	CatalogTupleUpdate(rel, &tup->t_self, tup);

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