Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail - Mailing list pgsql-hackers

From Selena Deckelmann
Subject Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail
Date
Msg-id 49B69053.6040601@endpoint.com
Whole thread Raw
In response to Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail  (Simon Riggs <simon@2ndQuadrant.com>)
List pgsql-hackers
Simon Riggs wrote:
> On Tue, 2009-03-10 at 07:30 -0700, Selena Deckelmann wrote:

>> A thing that could be added, however, is reporting of all invalid (as
>> opposed to valid, but requires a restart to apply) parameters before
>> exiting. This change requires refactoring ProcessConfigFile() more
>> significantly, as the parsing and validity checks are done separately,
>> and are exited with gotos. :)
>>
>> I haven't tried to change this yet, but was planning to.
>
> Can't we just do a reload, but with doit = false?

That would also be cool.  Like an 'apachectl configtest'.

There are five places where a 'goto' is used in guc-file.l needs to be 
changed to make this work (in addition to adding an option to pg_ctl).

It should be pretty straightforward, and I'll have a look at it tonight.

-selena

-- 
Selena Deckelmann
End Point Corporation


pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail
Next
From: "Joshua D. Drake"
Date:
Subject: Re: Updates of SE-PostgreSQL 8.4devel patches (r1704)