From: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Heikki Linnakangas <heikki(dot)linnakangas(at)iki(dot)fi>, <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Check for GiST index tuples that don't fit on a page. |
Date: | 2014-10-05 07:06:23 |
Message-ID: | 5430EDEF.3050603@vmware.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 10/04/2014 05:10 PM, Alvaro Herrera wrote:
> Peter Eisentraut wrote:
>> On 10/4/14 12:59 AM, Tom Lane wrote:
>>> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>>>> We don't have portable support for %zu formats until 9.4. The
>>>> backpatches to 9.3 and earlier need to use %lu.
>>> Good point --- and something that seems like it'll be an ongoing gotcha.
>>> Have we got any buildfarm critters that could be configured to throw
>>> errors about this?
>> If this change had contained a test, it would have caught it.
True. I didn't add a test because the error message depends on BLCKSZ
and alignment. It also didn't seem like a very likely issue to reoccur
in the same form.
>> Alternatively, we'd need to catch compiler warnings on the build farm,
>> but if the installation is too old to recognize %zu, who knows what else
>> it'll warn (or not warn) about. Eventually, users of such systems tend
>> point these things out, but that could take a while.
>
> I compiled 9.0 yesterday and there are a lot of warnings with my current
> compiler; something like -Werror is right out, I think. Maybe we can
> save warnings separately and have the member turn yellow if there are
> any, or something like that?
Hmm. For this particular case, it would straightforward to add a step to
the buildfarm script to do "grep -r '%z' src". It might turn up false
positives, if there's a %z in comments or such, but it shouldn't be much
effort to maintain a list of exceptions to filter out the false positives.
- Heikki
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2014-10-05 13:49:31 | Re: pgsql: Refactor replication connection code of various pg_basebackup ut |
Previous Message | Heikki Linnakangas | 2014-10-05 07:00:57 | pgsql: Cannot rely on %z printf length modifier. |