Re: [PATCH] Add inline comments to the pg_hba_file_rules view - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: [PATCH] Add inline comments to the pg_hba_file_rules view
Date
Msg-id ZP5ELFdn28Hn1Hfi@paquier.xyz
Whole thread Raw
In response to [PATCH] Add inline comments to the pg_hba_file_rules view  (Jim Jones <jim.jones@uni-muenster.de>)
Responses Re: [PATCH] Add inline comments to the pg_hba_file_rules view
List pgsql-hackers
On Mon, Sep 04, 2023 at 12:54:15PM +0200, Jim Jones wrote:
> The patch slightly changes the test 004_file_inclusion.pl to accommodate the
> new column and the hba comments.
>
> Discussion: https://www.postgresql.org/message-id/flat/3fec6550-93b0-b542-b203-b0054aaee83b%40uni-muenster.de

Well, it looks like what I wrote a couple of days ago was perhaps
confusing:
https://www.postgresql.org/message-id/ZPHAiNp%2ByKMsa/vc%40paquier.xyz
https://www.postgresql.org/message-id/ZPE8A7EnUH+ax5kw@paquier.xyz

This patch touches hbafuncs.c and the system view pg_hba_file_rules,
but I don't think this stuff should touch any of these code paths.
That's what I meant in my second message: the SQL portion should be
usable for all types of configuration files, even pg_ident.conf and
postgresql.conf, and not only pg_hba.conf.  A new SQL function
returning a SRF made of the comments extracted and the line numbers
can be joined with all the system views of the configuration files,
like sourcefile and sourceline in pg_settings, etc.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Avoid a possible null pointer (src/backend/utils/adt/pg_locale.c)
Next
From: Richard Guo
Date:
Subject: Re: Oversight in reparameterize_path_by_child leading to executor crash