From: | Greg Smith <greg(at)2ndQuadrant(dot)com> |
---|---|
To: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: includeifexists in configuration file |
Date: | 2011-12-12 19:24:53 |
Message-ID: | 4EE65505.1080305@2ndQuadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 11/16/2011 10:19 AM, Robert Haas wrote:
> I haven't read the code yet, but just to get the bikeshedding started,
> I think it might be better to call this include_if_exists rather than
> running it together as one word.
>
What's going on, it's like this bikeshed just disappeared. I should
figure out how that happened so I can replicate it.
This naming style change sounds fine to me, and I just adopted it for
the updated configuration directory patch. That patch now rearranges
the documentation this feature modifies. This is a pretty trivial
feature I'm not real concerned about getting a review for. I'll update
this with the name change and appropriate rebased patch once some
decision has been made about that one; will just bounce this forward to
January if it's still here when the current CF starts closing in earnest.
--
Greg Smith 2ndQuadrant US greg(at)2ndQuadrant(dot)com Baltimore, MD
PostgreSQL Training, Services, and 24x7 Support www.2ndQuadrant.us
From | Date | Subject | |
---|---|---|---|
Next Message | Marti Raudsepp | 2011-12-12 19:32:52 | Re: [PATCH] Caching for stable expressions with constant arguments v3 |
Previous Message | Robert Haas | 2011-12-12 19:19:45 | Re: Command Triggers |