From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Robert Haas <rhaas(at)postgresql(dot)org> |
Cc: | pgsql-hackers(at)postgresql(dot)org, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> |
Subject: | Re: [COMMITTERS] pgsql: Implement table partitioning. |
Date: | 2016-12-20 17:22:14 |
Message-ID: | 20161220172214.xjgratgkhqhsp7gk@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Robert Haas wrote:
> Implement table partitioning.
I thought it was odd to use rd_rel->reloftype as a boolean in
ATExecAttachPartition, but apparently we do it elsewhere too, so let's
leave that complaint for another day.
What I also found off in the same function is that we use
SearchSysCacheCopyAttName() on each attribute and then don't free the
result, and don't ever use the returned tuple either. A simple fix, I
thought, just remove the "Copy" and add a ReleaseSysCache(). But then I
noticed this whole thing is rather strange -- why not pass a boolean
flag down to CreateInheritance() and from there to
MergeAttributesIntoExisting() to implement the check? That seems less
duplicative.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2016-12-20 17:51:32 | Re: pgsql: Add pg_sequence system catalog |
Previous Message | Peter Eisentraut | 2016-12-20 16:12:34 | Re: pgsql: Add pg_sequence system catalog |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2016-12-20 17:51:32 | Re: pgsql: Add pg_sequence system catalog |
Previous Message | Andrew Dunstan | 2016-12-20 17:17:49 | Re: pgstattuple documentation clarification |