Re: [PATCHES] Magic block for modules

Magnus Hagander wrote:
> > > 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.

I say make it an ERROR and we can relax it later.  If you make it a
warning, we might not hear about it.

--  Bruce Momjian   http://candle.pha.pa.us EnterpriseDB    http://www.enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +