Re: pgsql: Fix union for pgstat message types - Mailing list pgsql-committers

From Tomas Vondra
Subject Re: pgsql: Fix union for pgstat message types
Date
Msg-id 20190504200437.zqo3vna4lez44b6p@development
Whole thread Raw
In response to Re: pgsql: Fix union for pgstat message types  (Magnus Hagander <magnus@hagander.net>)
List pgsql-committers
On Sat, May 04, 2019 at 07:08:10PM +0200, Magnus Hagander wrote:
>   On Sat, May 4, 2019 at 5:54 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
>     Tomas Vondra <tomas.vondra@2ndquadrant.com> writes:
>     > On Wed, May 01, 2019 at 10:34:52AM +0000, Magnus Hagander wrote:
>     >> Fix union for pgstat message types
>
>     > Should the tempfile bit be backported? That message was added in 9.2.
>
>     It's just cosmetic I think.  The main point of having everything in
>     the union is to make sure the union's size/alignment are sufficient,
>     but the tempfile message isn't anywhere near creating an issue there.
>
>   That was my thought as well. It's really only needed when we add new
>   things, to make sure we don't make those mistakes again. And nobody should
>   be adding new pgstat messages in old branches. 

Fine with me, I don't think the backbranches are broken. OTOH it's a
minor change that would make the code more similar.

regards

-- 
Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services 



pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: pgsql: pg_verify_checksums: Fix message punctuation
Next
From: Tom Lane
Date:
Subject: pgsql: Fix style violations in syscache lookups.