From: | Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> |
---|---|
To: | Amit Langote <amitlangote09(at)gmail(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: a comment in joinrel.c: compute_partition_bounds() |
Date: | 2021-10-06 08:42:24 |
Message-ID: | CAPmGK15n0C_hksSOkPVmciGFYM6MuUja+yFszUZF4wg9La8NDQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Sep 24, 2021 at 4:20 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> On Fri, Sep 24, 2021 at 3:34 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> > I think there's a word missing in the following comment:
> >
> > /*
> > * See if the partition bounds for inputs are exactly the same, in
> > * which case we don't need to work hard: the join rel have the same
> > * partition bounds as inputs, and the partitions with the same
> > * cardinal positions form the pairs.
> >
> > ": the join rel have the same..." seems to be missing a "will".
> >
> > Attached a patch to fix.
>
> Good catch! Will fix.
Rereading the comment, I think it would be better to add “will” to the
second part “the partitions with the same cardinal positions form the
pairs” as well. Updated patch attached.
Best regards,
Etsuro Fujita
Attachment | Content-Type | Size |
---|---|---|
comment-missing-word-2.patch | application/octet-stream | 899 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2021-10-06 08:55:53 | Re: wrapping CF 2021-09 |
Previous Message | Etsuro Fujita | 2021-10-06 08:30:54 | Re: RfC entries in CF 2021-09 |