Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail

Поиск
Список
Период
Сортировка
От Simon Riggs
Тема Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail
Дата
Msg-id 1236697480.28644.27.camel@ebony.2ndQuadrant
обсуждение исходный текст
Ответ на Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail  (Selena Deckelmann <selena@endpoint.com>)
Ответы Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail  (Selena Deckelmann <selena@endpoint.com>)
Список pgsql-hackers
On Tue, 2009-03-10 at 07:30 -0700, Selena Deckelmann wrote:

> > Is it possible to check for parameters that have been changed, yet will
> > not be applied at reload?
> 
> This was already implemented! :) For example:
> 
> LOG:  attempted change of parameter "shared_buffers" ignored
> DETAIL:  This parameter cannot be changed after server start.
> LOG:  attempted change of parameter "max_prepared_transactions" ignored
> DETAIL:  This parameter cannot be changed after server start.

OK, had that at the back of my mind for a while I guess.

> 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?

-- Simon Riggs           www.2ndQuadrant.comPostgreSQL Training, Services and Support



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Simon Riggs
Дата:
Сообщение: Re: Prepping to break every past release...
Следующее
От: Selena Deckelmann
Дата:
Сообщение: Re: postgresql.conf: patch to have ParseConfigFile report all parsing errors, then bail