From: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> |
---|---|
To: | Jaime Casanova <jaime(dot)casanova(at)2ndquadrant(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Amit Langote <amitlangote09(at)gmail(dot)com>, Rajkumar Raghuwanshi <rajkumar(dot)raghuwanshi(at)enterprisedb(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Declarative partitioning - another take |
Date: | 2016-11-08 04:08:45 |
Message-ID: | 3accb6b3-7d15-32d0-1d50-32892587b2d5@lab.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Jaime,
On 2016/11/08 2:24, Jaime Casanova wrote:
> On 7 November 2016 at 12:15, Jaime Casanova
> <jaime(dot)casanova(at)2ndquadrant(dot)com> wrote:
>> On 28 October 2016 at 02:53, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> wrote:
>>>
>>> Please find attached the latest version of the patches
>>
>> Hi,
>>
>> I started to review the functionality of this patch, so i applied all
>> 9 patches. After that i found this warning, which i guess is because
>> it needs a cast.
>>
>
> oh! i forgot the warning
> """
> partition.c: In function ‘get_qual_for_list’:
> partition.c:1159:6: warning: assignment from incompatible pointer type
> or = makeBoolExpr(OR_EXPR, list_make2(nulltest2, opexpr), -1);
> ^
> """
This one I noticed too and have fixed.
>
> attached a list of the warnings that my compiler give me (basically
> most are just variables that could be used uninitialized)
Thanks a lot for spotting and reporting these. Will fix as appropriate.
Regards,
Amit
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2016-11-08 04:28:35 | Re: Parallel tuplesort (for parallel B-Tree index creation) |
Previous Message | Michael Paquier | 2016-11-08 03:48:25 | Re: Replace use malloc() & friend by memory contexts for plperl and pltcl |