pgsql: Don't copy PartitionBoundInfo in set_relation_partition_info. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Don't copy PartitionBoundInfo in set_relation_partition_info.
Date
Msg-id E1h7OjF-0001qU-L8@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Don't copy PartitionBoundInfo in set_relation_partition_info.

I (tgl) remain dubious that it's a good idea for PartitionDirectory
to hold a pin on a relcache entry throughout planning, rather than
copying the data or using some kind of refcount scheme.  However, it's
certainly the responsibility of the PartitionDirectory code to ensure
that what it's handing back is a stable data structure, not that of
its caller.  So this is a pretty clear oversight in commit 898e5e329,
and one that can cost a lot of performance when there are many
partitions.

Amit Langote (extracted from a much larger patch set)

Discussion: https://postgr.es/m/CA+TgmoY3bRmGB6-DUnoVy5fJoreiBJ43rwMrQRCdPXuKt4Ykaw@mail.gmail.com
Discussion: https://postgr.es/m/9d7c5112-cb99-6a47-d3be-cf1ee6862a1d@lab.ntt.co.jp

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/c8151e642368599dc77c4448e6bdc34cc8810475

Modified Files
--------------
src/backend/optimizer/util/plancat.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)


pgsql-committers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: pgsql: Make heap TID a tiebreaker nbtree index column.
Next
From: Peter Geoghegan
Date:
Subject: pgsql: Go back to suppressing foreign_data DETAIL test output.