Re: Unmatched test and comment in partition_join.sql regression test

From: Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Unmatched test and comment in partition_join.sql regression test
Date: 2019-12-16 08:19:36
Message-ID: CAPmGK16wf1QcW-KE6AFsqL6r=4wV4zBHgNcks1g3ujRd4oM53A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Dec 13, 2019 at 7:17 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> I noticed this in the regression test while polishing the PWJ-enhancement patch:
>
> -- partitionwise join can not be applied for a join between list and range
> -- partitioned tables
> EXPLAIN (COSTS OFF)
> SELECT t1.a, t1.c, t2.b, t2.c FROM prt1_n t1 FULL JOIN prt1 t2 ON (t1.c = t2.c);
>
> The test doesn't match the comment which precedes it, because both
> tables are range-partitioned as shown below.

> I think the test should be moved to a more appropriate place,

On second thought I changed my mind; we would not need to move that
test, so I refrained from doing so.

> Attached is a patch for that. The
> patch fixes another misplaced comment as well.

I pushed an updated version of the patch.

Best regards,
Etsuro Fujita

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2019-12-16 08:58:19 Re: Unix-domain socket support on Windows
Previous Message Paul Guo 2019-12-16 07:37:41 Questions about SyncRepWaitForLSN()