Re: Foreign Tables as Partitions

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: David Fetter <david(at)fetter(dot)org>
Cc: PG Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Foreign Tables as Partitions
Date: 2013-07-22 18:10:53
Message-ID: CA+TgmoYVSXvZx88vCrEgmVv_QpeUGzOMeqMEmT8RXC+yZqW-bw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jul 19, 2013 at 1:46 PM, David Fetter <david(at)fetter(dot)org> wrote:
>> This functionality was actually present in the original submission
>> for foreign tables. I ripped it out before commit because I didn't
>> think all of the interactions with other commands had been
>> adequately considered. But I think they did consider it to some
>> degree, which this patch does not.
>
> A ref to the patch as submitted & patch as applied would help a lot :)

The patch as committed is 0d692a0dc9f0e532c67c577187fe5d7d323cb95b. I
suspect you can find the patch as submitted by going and figuring out
which CommitFest that was part of and working it out from there.

> Were there any particular things you managed to break with the patch
> as submitted? Places you thought would be soft but didn't poke at?

I think there was *some* discussion of this at the time. The basic
issue was to make sure all the commands that could encounter foreign
tables with this change but not without it behaved sanely. This would
include all variants of ALTER TABLE as well as ANALYZE and any other
utility commands that might view it as within their remit to recurse
to child tables.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavan Deolasee 2013-07-22 18:29:29 Expression indexes and dependecies
Previous Message David Fetter 2013-07-22 18:07:20 Re: [PATCH] pgbench --throttle (submission 7 - with lag measurement)