[HACKERS] Dropping a partitioned table takes too long - Mailing list pgsql-hackers

From Amit Langote
Subject [HACKERS] Dropping a partitioned table takes too long
Date
Msg-id 8f6f6519-a807-7bf1-acf7-f045da4ce385@lab.ntt.co.jp
Whole thread Raw
Responses Re: [HACKERS] Dropping a partitioned table takes too long  (高增琦 <pgf00a@gmail.com>)
Re: [HACKERS] Dropping a partitioned table takes too long  (Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>)
List pgsql-hackers
$SUBJECT, if the table has, say, 2000 partitions.

The main reason seems to be that RelationBuildPartitionDesc() will be
called that many times within the same transaction, which perhaps we
cannot do much about right away.  But one thing we could do is to reduce
the impact of memory allocations it does.  They are currently leaked into
the caller's context, which may not be reset immediately (such as
PortalHeapMemory).  Instead of doing it in the caller's context, use a
temporary context that is deleted before returning.  Attached is a patch
for that.  On my local development VM, `drop table
table_with_2000_partitions` finished in 27 seconds with the patch instead
of more than 20 minutes that it currently takes.

Thoughts?

Adding this to the open items list.

Thanks,
Amit

PS: this was actually mentioned by Ragnar Ouchterlony who reported some
bugs back in declarative partitioning in January [1]

[1]
https://www.postgresql.org/message-id/17d89e08-874b-c1b1-aa46-12d5afb26235%40agama.tv

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Attachment

pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: [HACKERS] StandbyRecoverPreparedTransactions recovers subtranslinks incorrectly
Next
From: David Rowley
Date:
Subject: Re: [HACKERS] Incorrect use of ERRCODE_UNDEFINED_COLUMN in extended stats