Re: Trigger regression test output

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: Lamar Owen <lamar(dot)owen(at)wgcr(dot)org>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Trigger regression test output
Date: 2002-10-03 17:06:37
Message-ID: 200210031706.g93H6bF14743@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane wrote:
> I said:
> > I am inclined to have the refint.c code emit the notice unconditionally
> > at DEBUG1 level, and then add a "SET client_min_messages = DEBUG1" in
> > the triggers regression test to ensure the notice will appear.
>
> Hmm, that doesn't look that good after all: the SET causes the
> regression output to be cluttered with a whole *lot* of chatter,
> which will doubtless change constantly and break the test regularly.
>
> Plan B is to make the refint.c code emit the message at NOTICE level,
> but to change the contrib makefile so that REFINT_VERBOSE is defined
> by default (ie, you gotta edit the makefile if you don't want it).
> This will work nicely for the regression tests' purposes. If there is
> anyone out there actually using refint.c in production, they might be
> annoyed by the NOTICE chatter, but quite honestly I doubt anyone is ---
> this contrib module has long since been superseded by standard
> foreign-key support.

Yes, but if few people are using it, should we question whether it
belongs in the standard regression tests at all?

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Justin Clift 2002-10-03 17:06:53 Re: Anyone want to assist with the translation of theAdvocacy
Previous Message Tino Wildenhain 2002-10-03 17:05:33 Re: Anyone want to assist with the translation of the