> >     ForeignPath *pathnode = makeNode(ForeignPath);
> > +   Assert(rel->rtekind == RTE_RELATION);
> >   
> >     pathnode->path.pathtype = T_ForeignScan;
> Maybe I'm missing the point, but I don't think it'd be better to put the
> assertion in create_foreignscan_path().  And I think it'd be the caller'
> responsiblity to ensure that equality, as any other pathnode creation
> routine for a baserel in pathnode.c assumes that equality.

Hmm. The assertion (not shown above but you put in
parameterize_path:) seems to say that 'base relation for foreign
paths must be a RTE_RELATION' isn't right? But I don't see
anything putting such a restriction in reparameterize_path
itself. Could you tell me where such a restriction comes from? Or
who needs such a restriction? I think any assertions shouldn't be
anywhere other than where just before needed.



Kyotaro Horiguchi
NTT Open Source Software Center

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to