Re: proposal: a validator for configuration files - Mailing list pgsql-hackers

From Robert Haas
Subject Re: proposal: a validator for configuration files
Date
Msg-id BANLkTi=U7DWt+ScCpfR3-iToQgnE6ZZ1Jg@mail.gmail.com
Whole thread Raw
In response to Re: proposal: a validator for configuration files  (Alexey Klyukin <alexk@commandprompt.com>)
Responses Re: proposal: a validator for configuration files
List pgsql-hackers
On Mon, Apr 4, 2011 at 2:03 PM, Alexey Klyukin <alexk@commandprompt.com> wrote:
> Here's the update of Selena's patch, which also shows all errors in
> configuration parameters (as well as parser errors) during reload.

You should add this here:

https://commitfest.postgresql.org/action/commitfest_view/open

On a quick glance, this patch appears to contain some superfluous
hunks where you changed whitespace or variable names.  You might want
to remove those and repost before adding to the CF app.  Also, some
submission notes would be very helpful - when you send in the revised
version, detail in the email the exact purpose of the changes so that
someone can review the patch without having to read this thread and
all preceding threads in their entirety.

Thanks,

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Proposal for GSoC : ADJ dashboard (Administration related software)
Next
From: Robert Haas
Date:
Subject: Re: WIP: Allow SQL-language functions to reference parameters by parameter name