Re: notice about costly ri checks (2)

From: Fabien COELHO <fabien(dot)coelho(at)ensmp(dot)fr>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: notice about costly ri checks (2)
Date: 2004-03-08 11:53:13
Message-ID: Pine.LNX.4.58.0403081245510.26010@sablons.cri.ensmp.fr
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches


> > As a side question, if there are multiple cross-type conversions in one
> > constraint on different column pairs, what do we think the message should
> > be? One message with multiple column mentions in detail or multiple
> > notices? (I haven't looked at the patch to see if one or the other is
> > easier with how it's set up)
>
> I would expect it to generate one WARNING for each mismatch; doing
> anything else would make life a lot more complex, both as to writing the
> code and as to formatting the output readably.

I agree.

patch v1 issued the warning once for each mismatch, in the check loop.

patch v2 issued the warning once for the foreign key, outside of the loop.

patch v3 is yet to come;-)

I'll re-submit a patch some time later, with a WARNING and mismatch
column names and types specified.

--
Fabien.

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message John Gray 2004-03-08 12:41:41 Re: contrib/xml2 function name change and minor bitrot
Previous Message Claudio Natoli 2004-03-08 09:14:40 pgstat EXEC_BACKEND