Re: hba load error and silent mode

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: hba load error and silent mode
Date: 2009-08-24 19:23:30
Message-ID: 9837222c0908241223q4b73d35ma8c1b8179c1e2c21@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Aug 24, 2009 at 20:51, Andrew Dunstan<andrew(at)dunslane(dot)net> wrote:
>
>
> Tom Lane wrote:
>>
>> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>>
>>>
>>> Tom Lane wrote:
>>>
>>>>
>>>> Oh, you mean move load_hba *down*, past the syslogger startup?
>>>> Yeah, that would probably be all right.
>>>>
>>
>>
>>>
>>> Well, that's what I originally said, yes ;-)
>>>
>>
>>
>>>
>>> But I don't think that precludes your more general suggestion regarding
>>> startup errors. In particular, I think moving the hba load down would be
>>> reasonable to backpatch to 8.4, whereas I doubt the general fix would.
>>>
>>
>> Well, the change I had in mind is only a few lines of code, and is
>> fixing a behavior that you yourself are arguing is unusably broken.
>> It seems like a reasonable back-patch candidate to me if we think this
>> is a serious bug.  But I personally wasn't seeing any of this as due for
>> back-patching.  The -S behavior has been like it is since forever, and
>> nobody's complained before.
>>
>>
>>
>
> We didn't check HBA validity at startup time before, did we? I would not be
> surprised to get more complaints now.

We checked some of it, but we check it a whole lot more now.

+1 for backpatching at least the move of the load_hba call.

--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message David Fetter 2009-08-24 19:27:43 Re: Bug in date arithmetic
Previous Message Pavel Stehule 2009-08-24 19:19:23 Re: Issues for named/mixed function notation patch