Re: [COMMITTERS] pgsql: Add support for coordinating record typmodsamong parallel worke - Mailing list pgsql-committers

From Thomas Munro
Subject Re: [COMMITTERS] pgsql: Add support for coordinating record typmodsamong parallel worke
Date
Msg-id CAEepm=08r6ssWnM7wOAN=omz=s9DHujsJjJY6SN_PTHayPR0hw@mail.gmail.com
Whole thread Raw
In response to Re: [COMMITTERS] pgsql: Add support for coordinating record typmods among parallel worke  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [COMMITTERS] pgsql: Add support for coordinating record typmods among parallel worke  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: [COMMITTERS] pgsql: Add support for coordinating record typmods among parallel worke  (Andres Freund <andres@anarazel.de>)
List pgsql-committers
On Fri, Sep 15, 2017 at 3:18 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Andres Freund <andres@anarazel.de> writes:
>> Add support for coordinating record typmods among parallel workers.
>
> Buildfarm not happy ...

My compiler, C++ and more recent C standards are OK with identical
redefinition of a typedef like that, but not the older standards or
those particular compilers.  D'oh.  (I should figure out how to make
my automatic patch tester this fussy).  I think we should probably
just do this:

diff --git a/src/include/access/session.h b/src/include/access/session.h
index 8376dc53127..910a9815d78 100644
--- a/src/include/access/session.h
+++ b/src/include/access/session.h
@@ -13,9 +13,7 @@#define SESSION_H
#include "lib/dshash.h"
-
-/* Defined in typcache.c */
-typedef struct SharedRecordTypmodRegistry SharedRecordTypmodRegistry;
+#include "utils/typcache.h"
/* * A struct encapsulating some elements of a user's session.  For now this

-- 
Thomas Munro
http://www.enterprisedb.com


-- 
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [COMMITTERS] pgsql: Add support for coordinating record typmods among parallel worke
Next
From: Tom Lane
Date:
Subject: Re: [COMMITTERS] pgsql: Add support for coordinating record typmods among parallel worke