Zoltan Boszormenyi wrote:
> 1. to minimize the already huge code impact on the relation case.
> 2. the checks done for the SELECT case is not exactly the
> same as for the realation case.
So put them in CopyToRelation. But the ones that apply to both, leave
in CopyTo.
> 3. the relation case is managed by passing around
> a Relation pointer, e.g. CopyGetAttnums. This simply
> not appropriate for the SELECT case.
Which is why you should leave the relation case alone and only add the
different case. The relation case is already known to be good.
--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support