From: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> |
---|---|
To: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Re: [COMMITTERS] pgsql: Use MINVALUE/MAXVALUE instead of UNBOUNDED for range partition b |
Date: | 2017-09-14 09:01:36 |
Message-ID: | 12e86064-6c2d-a812-b85a-8fec63f0b526@lab.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On 2017/09/14 16:53, Dean Rasheed wrote:
> On 13 September 2017 at 10:05, Amit Langote
> <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> wrote:
>> Coincidentally, I just wrote the patch for canonicalizing stored values,
>> instead of erroring out. Please see attached if that's what you were
>> thinking too.
>>
>
> Looks reasonable to me, if we decide to go this way.>
> One minor review comment --
Thanks for the review.
> it isn't really necessary to have the
> separate new boolean local variables as well as the datum kind
> variables. Just tracking the datum kind is sufficient and slightly
> simpler. That would also address a slight worry I have that your
> coding might result in a compiler warning about the kind variables
> being used uninitialised with some less intelligent compilers, not
> smart enough to work out that it can't actually happen.
Got rid of the boolean variables in the updated patch.
Thanks,
Amit
Attachment | Content-Type | Size |
---|---|---|
0001-Canonicalize-catalog-representation-of-range-partiti.patch | text/plain | 10.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-09-14 09:06:18 | pgsql: Properly check interrupts in execScan.c. |
Previous Message | Dean Rasheed | 2017-09-14 08:01:46 | Re: [COMMITTERS] pgsql: Use MINVALUE/MAXVALUE instead of UNBOUNDED for range partition b |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-09-14 09:06:41 | Re: uninterruptible state in 10beta4 |
Previous Message | amul sul | 2017-09-14 08:58:49 | Re: [POC] hash partitioning |