From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>,Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-committers <pgsql-committers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Add support for coordinating record typmods among parallel worke |
Date: | 2017-09-15 04:11:58 |
Message-ID: | EE83DBB6-815E-4D06-81D8-5A56B9604C09@anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On September 14, 2017 8:38:09 PM PDT, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> wrote:
>On Fri, Sep 15, 2017 at 3:18 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Andres Freund <andres(at)anarazel(dot)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
Sorry for missing that during review - unfortunately I don't have a computer with me now - so I won't get around to this till tomorrow...
Andres
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2017-09-15 04:20:07 | Re: [COMMITTERS] pgsql: passwordcheck: Add test suite |
Previous Message | Tom Lane | 2017-09-15 03:45:20 | Re: pgsql: Add support for coordinating record typmods among parallel worke |
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2017-09-15 04:20:07 | Re: [COMMITTERS] pgsql: passwordcheck: Add test suite |
Previous Message | Tom Lane | 2017-09-15 03:45:20 | Re: pgsql: Add support for coordinating record typmods among parallel worke |