From: | Amit Langote <amitlangote09(at)gmail(dot)com> |
---|---|
To: | Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> |
Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Obsolete comment in partbounds.c |
Date: | 2019-10-23 03:08:04 |
Message-ID: | CA+HiwqExXpfyqK1xNuTY9eo4N-pLXviP7AxTpdYB2bhozN6USg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Oct 21, 2019 at 5:44 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> On Sat, Oct 19, 2019 at 5:56 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> > On Fri, Oct 18, 2019 at 6:56 PM Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> wrote:
> > > On 2019-Oct-18, Etsuro Fujita wrote:
> > > > While reviewing the partitionwise-join patch, I noticed $Subject,ie,
> > > > this in create_list_bounds():
> > > >
> > > > /*
> > > > * Never put a null into the values array, flag instead for
> > > > * the code further down below where we construct the actual
> > > > * relcache struct.
> > > > */
> > > > if (null_index != -1)
> > > > elog(ERROR, "found null more than once");
> > > > null_index = i;
> > > >
> > > > "the code further down below where we construct the actual relcache
> > > > struct" isn't in the same file anymore by refactoring by commit
> > > > b52b7dc25. How about modifying it like the attached?
> > >
> > > Yeah, agreed. Instead of "the null comes from" I would use "the
> > > partition that stores nulls".
> >
> > I think your wording is better than mine. Thank you for reviewing!
>
> I applied the patch down to PG12.
Thank you Fujita-san and Alvaro.
Regards,
Amit
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Langote | 2019-10-23 03:28:24 | Re: d25ea01275 and partitionwise join |
Previous Message | Amit Langote | 2019-10-23 02:59:45 | Re: [PATCH] use separate PartitionedRelOptions structure to store partitioned table options |