]> granicus.if.org Git - postgresql/commit
Get rid of copy_partition_key
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Thu, 21 Dec 2017 17:21:39 +0000 (14:21 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Thu, 21 Dec 2017 17:21:39 +0000 (14:21 -0300)
commit8a0596cb656e357c391cccf12854beb2e05f3901
treecde4a211405b9bcb1406ec8cb28ab3ab400a1068
parent9ef6aba1d3513829d9f77a4a91ca52f2e5719aef
Get rid of copy_partition_key

That function currently exists to avoid leaking memory in
CacheMemoryContext in case of trouble while the partition key is being
built, but there's a better way: allocate everything in a memcxt that
goes away if the current (sub)transaction fails, and once the partition
key is built and no further errors can occur, make the memcxt permanent
by making it a child of CacheMemoryContext.

Reviewed-by: Tom Lane
Discussion: https://postgr.es/m/20171027172730.eh2domlkpn4ja62m@alvherre.pgsql
src/backend/utils/cache/relcache.c