From: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: automating pg_config.h.win32 maintenance |
Date: | 2019-12-20 08:17:14 |
Message-ID: | 1361e2a4-327c-78d2-40bf-027891c88982@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2019-12-19 08:49, Michael Paquier wrote:
> On Thu, Dec 19, 2019 at 08:31:05AM +0100, Peter Eisentraut wrote:
>> On 2019-12-19 04:59, Michael Paquier wrote:
>>> This part needs a comment. Like it is the equivalent of what
>>> src/common/'s Makefile does or something like that?
>>
>> This was meant to be addressed by <https://www.postgresql.org/message-id/flat/6e457870-cef5-5f1d-b57c-fc89cfb8a788%402ndquadrant.com>,
>> but that discussion has not concluded yet. Perhaps it makes more sense in
>> this context.
>
> Hmm. Your patch does not really change the generation of
> VAL_CONFIGURE in pg_config.h, so I am not sure that this other thread
> is an actual barrier for the improvement discussed here. I would be
> actually fine to just remove the XXX and still use GetFakeConfigure()
> with VAL_CONFIGURE for now. It would be a good thing to get rid of
> pg_config.h.win32 definitely.
committed with that comment removed
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Kyotaro Horiguchi | 2019-12-20 08:29:21 | Re: PATCH: logical_work_mem and logical streaming of large in-progress transactions |
Previous Message | Tatsuo Ishii | 2019-12-20 08:10:58 | Re: Implementing Incremental View Maintenance |