On Fri, Dec 20, 2013 at 1:05 PM, Sawada Masahiko <sawada.m...@gmail.com> wrote:
> On Fri, Dec 20, 2013 at 3:38 AM, Sawada Masahiko <sawada.m...@gmail.com> 
> wrote:
>> On Thu, Dec 19, 2013 at 12:37 PM, Amit Kapila <amit.kapil...@gmail.com> 
>> wrote:
>>> On Wed, Dec 18, 2013 at 11:30 AM, Michael Paquier
>>> <michael.paqu...@gmail.com> wrote:
>>>> On Wed, Dec 18, 2013 at 11:22 AM, Amit Kapila <amit.kapil...@gmail.com> 
>>>> wrote:
>>>>> On Fri, Dec 13, 2013 at 7:57 PM, Heikki Linnakangas
>>>>> <hlinnakan...@vmware.com> wrote:
>>>>>> Thanks, committed with some minor changes:
>>>>> Should this patch in CF app be moved to Committed Patches or is there
>>>>> something left for this patch?
>>>> Nothing has been forgotten for this patch. It can be marked as committed.
>>> Thanks for confirmation, I have marked it as Committed.
>> Thanks!
>> I attached the patch which changes name from 'wal_log_hintbits' to
>> 'wal_log_hints'.
>> It gained the approval of plural.
> Sorry the patch which I attached has wrong indent on pg_controldata.
> I have modified it and attached the new version patch.
Now that you send this patch, I am just recalling some recent email
from Tom arguing about avoiding to mix lower and upper-case characters
for a GUC parameter name:

To fullfill this requirement, could you replace walLogHints by
wal_log_hints in your patch? Thoughts from others?

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to