Re: Teaching pg_dump to use NOT VALID constraints

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Teaching pg_dump to use NOT VALID constraints
Date: 2014-11-10 18:00:22
Message-ID: CA+U5nM+cucBdTXTjPqtLPxRK=XveLWo3PmJ8MYmUXDiJvY9Cdg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 10 November 2014 17:33, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> wrote:

>> pg_dump --no-revalidaton
>>
>> will add "NOT VALID" onto the recreation SQL for any FKs, but only for
>> ones that were already known to be valid.
>
> Well. Constraints that haven't been validated already have a NOT VALID
> emitted by ruleutils.c, yes? So what this patch does is add such a
> clause for all *other* constraints. Right? In other words what it aims
> to do is speed up loading of data by skipping the validation step on
> restore. Is that right?

Correct. CHECK constraints are added onto main table so they validate at load.

> ISTM we could have the default pg_dump behavior emit NOT VALID
> constraints, and add VALIDATE CONSTRAINT commands at the end; that way
> the database is usable sooner but the constraints end up marked as
> validated by the time the dump is finished.

Yes, may be an even better idea. We'd still want the --no-revalidation
option, AFAICS.

FKs are already "at the end". Perhaps we should add another
"validation" section?

I like the idea, just not sure how long it would take.

--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jim Nasby 2014-11-10 18:03:59 Re: Proposal: Log inability to lock pages during vacuum
Previous Message Joshua D. Drake 2014-11-10 17:50:55 Re: remove pg_standby?