pgsql: Improve ExecEvalVar's handling of whole-row variables in cases - Mailing list pgsql-committers

From tgl@postgresql.org (Tom Lane)
Subject pgsql: Improve ExecEvalVar's handling of whole-row variables in cases
Date
Msg-id 20100111153104.42E847541B9@cvs.postgresql.org
Whole thread Raw
List pgsql-committers
Log Message:
-----------
Improve ExecEvalVar's handling of whole-row variables in cases where the
rowtype contains dropped columns.  Sometimes the input tuple will be formed
from a select targetlist in which dropped columns are filled with a NULL
of an arbitrary type (the planner typically uses INT4, since it can't tell
what type the dropped column really was).  So we need to relax the rowtype
compatibility check to not insist on physical compatibility if the actual
column value is NULL.

In principle we might need to do this for functions returning composite
types, too (see tupledesc_match()).  In practice there doesn't seem to be
a bug there, probably because the function will be using the same cached
rowtype descriptor as the caller.  Fixing that code path would require
significant rearrangement, so I left it alone for now.

Per complaint from Filip Rembialkowski.

Modified Files:
--------------
    pgsql/src/backend/executor:
        execQual.c (r1.260 -> r1.261)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execQual.c?r1=1.260&r2=1.261)

pgsql-committers by date:

Previous
From: adunstan@postgresql.org (Andrew Dunstan)
Date:
Subject: pgsql: remove use of temp file for perlchunks.h, as in recent change to
Next
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql: Improve ExecEvalVar's handling of whole-row variables in cases