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.
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 |