Re: [PATCHES] Magic block for modules - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: [PATCHES] Magic block for modules
Date
Msg-id 6BCB9D8A16AC4241919521715F4D8BCEA35461@algol.sollentuna.se
Whole thread Raw
Responses Re: [PATCHES] Magic block for modules
List pgsql-hackers
> > On Sun, May 07, 2006 at 08:21:43PM -0400, Tom Lane wrote:
> >> I'm pretty sure we had agreed that magic blocks should be
> required;
> >> otherwise this check will accomplish little.
>
> > Sure, I just didn't want to break every module in one
> weekend. I was
> > thinking of adding it with LOG level now, send a message on
> -announce
> > saying that at the beginning of the 8.2 freeze it will be an ERROR.
> > Give people time to react.
>
> Now that the magic-block patch is in, we need to revisit this
> bit of the discussion.  I'm for making lack of a magic block
> an ERROR immediately.
> I don't see the point of waiting; in fact, if we wait till
> freeze we'll just make the breakage more concentrated.  At
> the very least it ought to be a WARNING immediately, because
> a LOG message is just not visible enough.
>
> Comments?

If it's eventually going to be an ERROR, it's better to make it ERROR
from the start.

People working off cvs snapshot will (hopefully) expect temporary
breakage during the development period. In general, you'd expect less
breakage the closer to release you are.

//Magnus


pgsql-hackers by date:

Previous
From: "Dave Page"
Date:
Subject: Re: PQescapeIdentifier
Next
From: Bruce Momjian
Date:
Subject: Re: Compile libpq with vc8