From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
---|---|
To: | Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com> |
Cc: | Justin Pryzby <pryzby(at)telsasoft(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: ATTACH PARTITION locking documentation for DEFAULT partitions |
Date: | 2021-07-27 06:02:10 |
Message-ID: | CAApHDvr7PZmpDjN3Qrgv9YjXNaR0zAo_rLhyeYP1H2vUgwZ2Kw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, 13 Jul 2021 at 02:30, Matthias van de Meent
<boekewurm+postgres(at)gmail(dot)com> wrote:
> The algoritm as described in your patch implies that this recursive
> locking is conditional on _only_ the check-constraints of the topmost
> partition ("performed whilst holding ... and all of its
> sub-partitions, if any"), whereas actually the locking on each
> (sub-)partition is determined by the constraints of the hierarchy down
> to that child partition. It in actuality, this should not matter much,
> but this is a meaningful distinction that I wanted to call out.
I had in mind that was implied, but maybe it's better to be explicit about that.
I've adjusted the patch and attached what I came up with. Let me know
what you think.
I think this can be back-patched as far as 12. Before then we took an
AEL on the partitioned table, so it seems much less important since
any concurrency would be blown out by the AEL.
David
Attachment | Content-Type | Size |
---|---|---|
v2_partition_doc_fixes.patch | application/octet-stream | 3.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2021-07-27 06:05:19 | Re: ORDER BY pushdowns seem broken in postgres_fdw |
Previous Message | Corey Huinker | 2021-07-27 06:01:13 | Re: Fwd: Emit namespace in post-copy output |