Re: DOCS: add helpful partitioning links

From: Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>
To: Robert Treat <rob(at)xzilla(dot)net>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: DOCS: add helpful partitioning links
Date: 2024-03-25 10:43:10
Message-ID: CAExHW5u8tzDvL1S91-OPjVmF0uGpECHuAhsqJ-sH-5733VPgtw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Mar 22, 2024 at 10:58 PM Robert Treat <rob(at)xzilla(dot)net> wrote:

>
> v5 patch attached which I think further improves clarity/brevity of
> this section. I've left the patch name the same for simplicity, but
> I'd agree that the commit would now be more along the lines of editing
> / improvements / copyrighting of "Partition Maintenance" docs.
>

Right. Minor suggestions.

- It is recommended to drop the now-redundant <literal>CHECK</literal>
- constraint after the <command>ATTACH PARTITION</command> is
complete. If
- the table being attached is itself a partitioned table, then each of
its
+ As illustrated above, it is recommended to avoid this scan by
creating a
+ <literal>CHECK</literal> constraint on the to be attached table that

Instead of "to be attached table", "table to be attached" reads better. You
may want to add "as a partition" after that.

Similarly, if the partitioned table has a <literal>DEFAULT</literal>
partition, it is recommended to create a <literal>CHECK</literal>
constraint which excludes the to-be-attached partition's constraint.
If
- this is not done then the <literal>DEFAULT</literal> partition will be
+ this is not done, the <literal>DEFAULT</literal> partition must be

I am not sure whether replacing "will" by "must" is correct. Usually I have
seen "will" being used in such sentences, "must" seems appropriate given
the necessity.

--
Best Wishes,
Ashutosh Bapat

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ashutosh Bapat 2024-03-25 10:56:34 Re: Memory consumed by child SpecialJoinInfo in partitionwise join planning
Previous Message jian he 2024-03-25 10:40:30 Re: session username in default psql prompt?