Re: [COMMITTERS] pgsql: Fix field order in struct catcache.

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Fix field order in struct catcache.
Date: 2017-01-13 13:30:55
Message-ID: 20170113133055.xgc4jvtwbszs2475@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Tom Lane wrote:
> Fix field order in struct catcache.
>
> Somebody failed to grasp the point of having the #ifdef CATCACHE_STATS
> fields at the end of the struct. Put that back the way it should be,
> and add a comment making it more explicit why it should be that way.

Actually, the CATCACHE_STATS was added purposefully not at end of
struct, because back then cc_bucket was a variable-length array; see
commit 786340441706 (2002). It only became possible to do this after
commit 20cb18db4668 (2013). I researched this because I vaguely
believed this to be my fault (a66ee69ad) but that's not so.

--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2017-01-13 13:35:59 Re: [COMMITTERS] pgsql: Fix field order in struct catcache.
Previous Message Tom Lane 2017-01-13 00:00:07 pgsql: Fix field order in struct catcache.

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2017-01-13 13:35:59 Re: [COMMITTERS] pgsql: Fix field order in struct catcache.
Previous Message Tom Lane 2017-01-13 13:29:43 Re: Fixing matching of boolean index columns to sort ordering