Re: [PATCH] Add session_preload_libraries configuration parameter

Поиск
Список
Период
Сортировка
От Robins
Тема Re: [PATCH] Add session_preload_libraries configuration parameter
Дата
Msg-id CAEP4nAw8AzZ_36yK0T=4WAp9Z_jmGTxosnzsAVeu26BYLW=19w@mail.gmail.com
обсуждение исходный текст
Ответ на [PATCH] Add session_preload_libraries configuration parameter  (Peter Eisentraut <peter_e@gmx.net>)
Ответы Re: [PATCH] Add session_preload_libraries configuration parameter  (Robins <robins@pobox.com>)
Список pgsql-hackers
On 12 June 2013 22:12, Peter Eisentraut <peter_e@gmx.net> wrote:
This is like shared_preload_libraries except that it takes effect at
backend start and can be changed without a full postmaster restart.  It
is like local_preload_libraries except that it is still only settable by
a superuser.  This can be a better way to load modules such as
auto_explain.

Since there are now three preload parameters, regroup the documentation
a bit.  Put all parameters into one section, explain common
functionality only once, update the descriptions to reflect current and
future realities.
---

Hi,

Did some basic checks on this patch. List-wise feedback below.

- Cleanly applies to Git-Head: Yes (Minor 1 line offset in guc.c, but that's probably because of the delay in reviewing)
- Documentation Updated: Yes
- All tests pass: Yes
- Removed unnecessary extra-lines: Yes

- Do we want it?: ???

- Any visible issues: No
- Any compiler warnings: No

- Others: 
Number of new lines added not covered by tests: -107 (Wierd but, it seems to have reduced line coverage). But correct me if am not seeing the elephant in the room.

--
Robins Tharakan

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

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: pg_ctl and -h/help
Следующее
От: Fabrízio de Royes Mello
Дата:
Сообщение: Re: Patch to add support of "IF NOT EXISTS" to others "CREATE" statements