From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Jeevan Ladhe <jeevan(dot)ladhe(at)enterprisedb(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, amul sul <sulamul(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Bug in pg_dump --table and --exclude-table for declarative partition table handling. |
Date: | 2017-05-12 01:47:39 |
Message-ID: | CA+TgmoZiHcPCa_-Y-gcSv6_+6ZfADHAHyPx-7hoZ77QKWwL4uw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, May 11, 2017 at 9:33 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Uh ... what in that is creating the already-extant parent?
/me looks embarrassed.
Never mind. I didn't read what you wrote carefully enough.
>> I think one answer to the original complaint might be to add a new
>> flag to pg_dump, something like --recursive-selection, maybe -r for
>> short, which makes --table, --exclude-table, and --exclude-table-data
>> cascade to inheritance descendents.
>
> Yeah, you could do it like that. Another way to do it would be to
> create variants of all the selection switches, along the lines of
> "--table-all=foo" meaning "foo plus its children". Then you could
> have some switches recursing and others not within the same command.
> But maybe that's more flexibility than needed ... and I'm having a
> hard time coming up with nice switch names, anyway.
I don't think that's as good. It's a lot more typing than what I
proposed and I don't think anyone is really going to want the
flexibility.
> Anyway, I'm still of the opinion that it's fine to leave this as a
> future feature. If we've gotten away without it this long for
> inherited tables, it's unlikely to be critical for partitioned tables.
+1.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2017-05-12 01:54:48 | Re: Moving relation extension locks out of heavyweight lock manager |
Previous Message | Bossart, Nathan | 2017-05-12 01:43:48 | Re: [Proposal] Allow users to specify multiple tables in VACUUM commands |