Thread: Fix example in partitioning documentation

Fix example in partitioning documentation

From
Amit Langote
Date:
Hi,

As of v12, Append node is elided when there's a single subnode under
it.  An example in the partitioning documentation needs to be fixed to
account for that change.  Attached a patch.

Thanks,
Amit

Attachment

Re: Fix example in partitioning documentation

From
Michael Paquier
Date:
On Tue, Sep 24, 2019 at 10:52:30AM +0900, Amit Langote wrote:
> As of v12, Append node is elided when there's a single subnode under
> it.  An example in the partitioning documentation needs to be fixed to
> account for that change.  Attached a patch.

Indeed, using the same example as the docs:
CREATE TABLE measurement (
     logdate         date not null,
     peaktemp        int,
     unitsales       int
 ) PARTITION BY RANGE (logdate);
CREATE TABLE measurement_y2016m07
     PARTITION OF measurement (
     unitsales DEFAULT 0
 ) FOR VALUES FROM ('2016-07-01') TO ('2016-08-01');
SET enable_partition_pruning = on;
EXPLAIN SELECT count(*) FROM measurement WHERE logdate = DATE
'2016-07-02';

I'll take care of committing that, however this will have to wait
until v12 RC1 is tagged.
--
Michael

Attachment

Re: Fix example in partitioning documentation

From
Amit Langote
Date:
On Tue, Sep 24, 2019 at 11:14 AM Michael Paquier <michael@paquier.xyz> wrote:
> On Tue, Sep 24, 2019 at 10:52:30AM +0900, Amit Langote wrote:
> > As of v12, Append node is elided when there's a single subnode under
> > it.  An example in the partitioning documentation needs to be fixed to
> > account for that change.  Attached a patch.
>
> Indeed, using the same example as the docs:
> CREATE TABLE measurement (
>      logdate         date not null,
>      peaktemp        int,
>      unitsales       int
>  ) PARTITION BY RANGE (logdate);
> CREATE TABLE measurement_y2016m07
>      PARTITION OF measurement (
>      unitsales DEFAULT 0
>  ) FOR VALUES FROM ('2016-07-01') TO ('2016-08-01');
> SET enable_partition_pruning = on;
> EXPLAIN SELECT count(*) FROM measurement WHERE logdate = DATE
> '2016-07-02';
>
> I'll take care of committing that, however this will have to wait
> until v12 RC1 is tagged.

Sure, thank you.

Regards,
Amit



Re: Fix example in partitioning documentation

From
Michael Paquier
Date:
On Tue, Sep 24, 2019 at 11:36:40AM +0900, Amit Langote wrote:
> Sure, thank you.

And done with f5daf7f, back-patched down to 12.
--
Michael

Attachment

Re: Fix example in partitioning documentation

From
Amit Langote
Date:
On Wed, Sep 25, 2019 at 1:47 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Tue, Sep 24, 2019 at 11:36:40AM +0900, Amit Langote wrote:
> > Sure, thank you.
>
> And done with f5daf7f, back-patched down to 12.

Thanks again. :)

Regards,
Amit