Re: [PATCH] Add inline comments to the pg_hba_file_rules view

Поиск
Список
Период
Сортировка
От Peter Eisentraut
Тема Re: [PATCH] Add inline comments to the pg_hba_file_rules view
Дата
Msg-id 8313d028-9e8b-c657-81a5-c406ac4c44c3@eisentraut.org
обсуждение исходный текст
Ответ на [PATCH] Add inline comments to the pg_hba_file_rules view  (Jim Jones <jim.jones@uni-muenster.de>)
Ответы Re: [PATCH] Add inline comments to the pg_hba_file_rules view  (Daniel Gustafsson <daniel@yesql.se>)
Re: [PATCH] Add inline comments to the pg_hba_file_rules view  (Jim Jones <jim.jones@uni-muenster.de>)
Список pgsql-hackers
On 04.09.23 11:54, Jim Jones wrote:
> This patch proposes the column "comment" to the pg_hba_file_rules view. 
> It basically parses the inline comment (if any) of a valid pg_hba.conf 
> entry and displays it in the new column.
> 
> For such pg_hba entries ...
> 
> host db jim 127.0.0.1/32 md5 # foo
> host db jim 127.0.0.1/32 md5 #bar
> host db jim 127.0.0.1/32 md5 #     #foo#

I'm skeptical about this.

First, there are multiple commenting styles.  The end-of-line style is 
less common in my experience, because pg_hba.conf lines tend to belong. 
Another style is

# foo
host db jim 127.0.0.1/32 md5
# bar
host db jim 127.0.0.1/32 md5

or even as a block

# foo and bar
host db jim 127.0.0.1/32 md5
host db jim 127.0.0.1/32 md5

Another potential problem is that maybe people don't want their comments 
leaked out of the file.  Who knows what they have written in there.

I think we should leave file comments be file comments.  If we want some 
annotations to be exported to higher-level views, we should make that an 
intentional and explicit separate feature.




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

Предыдущее
От: Alexander Pyhalov
Дата:
Сообщение: Re: Partial aggregates pushdown
Следующее
От: "Hayato Kuroda (Fujitsu)"
Дата:
Сообщение: RE: [PGdocs] fix description for handling pf non-ASCII characters