Re: Re: pg_hba.conf pre-parsing change

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Re: pg_hba.conf pre-parsing change
Date: 2001-07-31 23:39:15
Message-ID: 200107312339.f6VNdFi20790@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> >> Would it make sense to do fstat calls on these files and reload whenever
> >> we observe that the file modification time has changed? That'd be an
> >> additional kernel call per connection attempt, so I'm not at all sure
> >> I want to do it ... but it ought to be debated. Comments anyone?
>
> > We could, but we don't with postgresql.conf so it made sense to keep the
> > behavior the same for the two files.
>
> I'm inclined to agree --- for one thing, this allows one to edit the
> files in place without worrying that the postmaster will pick up a
> partially-edited file. But I wanted to throw the issue out to pghackers
> to see if anyone would be really unhappy about having to SIGHUP the
> postmaster after changing the authorization conf files.

OK.

> In any case, if we don't change the code, the change in behavior from
> prior releases needs to be documented...

You mean in the SGML or in the release highlight text?

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Lamar Owen 2001-07-31 23:46:57 Re: Re: pg_hba.conf pre-parsing change
Previous Message Tom Lane 2001-07-31 23:20:14 Re: pg_hba.conf pre-parsing change

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2001-07-31 23:42:37 Re: [PATCHES] Allow IDENT authentication on local connections (Linux only)
Previous Message Tom Lane 2001-07-31 23:39:05 Re: [PATCHES] Allow IDENT authentication on local connections (Linux only)