Re: pg_hba_file_settings view patch

Поиск
Список
Период
Сортировка
От Haribabu Kommi
Тема Re: pg_hba_file_settings view patch
Дата
Msg-id CAJrrPGfPiBRyibqy27Gnzcpvy0WUtwRNrGDsayuZPV9eG9uVow@mail.gmail.com
обсуждение исходный текст
Ответ на Re: pg_hba_file_settings view patch  (Simon Riggs <simon@2ndquadrant.com>)
Ответы Re: pg_hba_file_settings view patch  (Michael Paquier <michael.paquier@gmail.com>)
Список pgsql-hackers


On Sun, Sep 4, 2016 at 1:44 AM, Simon Riggs <simon@2ndquadrant.com> wrote:
On 15 August 2016 at 12:17, Haribabu Kommi <kommi.haribabu@gmail.com> wrote:

> comments?

This looks like a good feature contribution, thanks.

At present the patch doesn't apply cleanly, please rebase.

Rebased patch is attached.
 
The patch doesn't contain any tests, which means I can't see what the
output looks like, so I can't judge the exact usefulness of this
particular patch. ISTM likely that there will be some detailed points
to review in there somewhere.

Added a test in the regress and also in the docs.

Do we want line number, or priority order? i.e. do we want to count
comment lines or just main rule lines? I prefer latter.
Various other questions along those lines needed, once I can see the output.

I preferred the line number that includes the comment lines also. This way it
will be easy to edit the file if it contains any errors by directly going to that line
number.
 
What is push_jsonb_string_value() etc..?
If there is required infrastructure there is no explanation of why.
Suggest you explain and/or split into two.

I added a JSONB type column to display the hba.conf options values.
To store the options data into JSONB format, currently i didn't find any
functions that are available to use in the core. So I added key/value
functions to add the data into JSONB object.

The functions related code is split into a different patch and attached.
 
pg_hba_file_settings seems a clumsy name. I'd prefer pg_hba_settings,
since that name could live longer than the concept of pg_hba.conf,
which seems likely to become part of ALTER SYSTEM in future, so we
wouldn't really want the word "file" in there.

Yes, I also agree that *file* is not required. The hba rules are not available
in memory also in the backends. I changed the view name to pg_hba_rules
as per the other mail from Christoph.


Regards,
Hari Babu
Fujitsu Australia
Вложения

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

Предыдущее
От: Heikki Linnakangas
Дата:
Сообщение: Re: Better locale-specific-character-class handling for regexps
Следующее
От: Christian Ullrich
Дата:
Сообщение: Re: Parallel build with MSVC