Re: Minor inheritance/check bug: Inconsistent behavior

From: Amit Kapila <amit(dot)kapila(at)huawei(dot)com>
To: "'Tom Lane'" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "'Robert Haas'" <robertmhaas(at)gmail(dot)com>
Cc: "'Chris Travers'" <chris(at)metatrontech(dot)com>, <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: Minor inheritance/check bug: Inconsistent behavior
Date: 2012-08-28 10:40:07
Message-ID: 004a01cd8509$7e2f95a0$7a8ec0e0$@kapila@huawei.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

From: Tom Lane [mailto:tgl(at)sss(dot)pgh(dot)pa(dot)us]
Sent: Tuesday, August 28, 2012 2:04 AM
Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> Maybe, but in that case shouldn't referencing a system column chuck an
error?

> Yeah, possibly. I think none of them are populated with anything useful
> during INSERT checks (though OID might be an exception?). Less sure
> about the state during UPDATE checks, though.

AFAICT during Update also, it doesn't contain useful. The only chance it
would have contain something useful is when it goes for EvalPlanQual and
then again comes to check for constraints. However these attributes get
filled in heap_update much later.

So now should the fix be that it returns an error for system column
reference except for OID case?

With Regards,
Amit Kapila.

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Ryan Kelly 2012-08-28 11:40:07 Re: [PATCH] Prevent hanging on unreachable hosts on startup
Previous Message Chris Travers 2012-08-28 06:46:13 Re: Minor inheritance/check bug: Inconsistent behavior