Re: PGDLLEXPORTing all GUCs? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: PGDLLEXPORTing all GUCs?
Date
Msg-id 27201.1399472976@sss.pgh.pa.us
Whole thread Raw
In response to Re: PGDLLEXPORTing all GUCs?  (Andres Freund <andres@2ndquadrant.com>)
Responses Re: PGDLLEXPORTing all GUCs?
Re: PGDLLEXPORTing all GUCs?
List pgsql-hackers
Andres Freund <andres@2ndquadrant.com> writes:
> On 2014-05-07 09:35:06 -0400, Tom Lane wrote:
>> Craig Ringer <craig@2ndquadrant.com> writes:
>>> Is there any reason _not_ to PGDLLEXPORT all GUCs, other than cosmetic
>>> concerns?

>> That seems morally equivalent to "is there a reason not to make every
>> static variable global?".

> I think what Craig actually tries to propose is to mark all GUCs
> currently exported in headers PGDLLIMPORT.

There are few if any GUCs that aren't exposed in headers, just so that
guc.c can communicate with the owning modules.  That doesn't mean that
we want everybody in the world messing with them.

To my mind, we PGDLLEXPORT some variable only after deciding that yeah,
we're okay with having third-party modules touching that.  Craig's
proposal is to remove human judgement from that process.
        regards, tom lane



pgsql-hackers by date:

Previous
From: Craig Ringer
Date:
Subject: Re: PGDLLEXPORTing all GUCs?
Next
From: Andrew Dunstan
Date:
Subject: Re: proposal: Set effective_cache_size to greater of .conf value, shared_buffers