Daniel Gustafsson <daniel@yesql.se> writes:
>> On 4 May 2026, at 21:22, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> As a side matter, "load_host" seems like a remarkably generic name
>> that conveys little about what it actually does,
> It's following the naming convention of load_hba() which reads pg_hba.conf, and
> load_ident() which reads pg_ident.conf - thus load_hosts() for the function
> that reads pg_hosts.conf. Perhaps load_pg_hosts_conf() or load_hosts_config()
> would convey more meaning?
Hmm, okay. I'd prefer a more specific name, but it wouldn't make
much sense unless we also rename those two. That's probably more
code churn than is justified.
I'll push the thing for moving/using the typedef, but leave the
function name alone.
regards, tom lane
