Fix breakage in AlterFunction().
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 20 Apr 2022 03:03:59 +0000 (23:03 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 20 Apr 2022 03:03:59 +0000 (23:03 -0400)
commitf583633bc130e8a572e5d29b9c8b2cc48885ecbf
tree9e90ded40724916475e5c2c0f30dbbbb6e1b134c
parent82d4a17a17508a4e6ff9038d81a7dfb9f19e104a
Fix breakage in AlterFunction().

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 7aea8e4f2.

(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
src/backend/commands/functioncmds.c