| 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: | Whole Thread | Raw Message | 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
| 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 |