Re: allow COPY routines to read arbitrary numbers of fields

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: allow COPY routines to read arbitrary numbers of fields
Date: 2010-12-06 18:23:36
Message-ID: 29152.1291659816@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> On 12/06/2010 12:11 PM, Tom Lane wrote:
>> Also, why did you change the setup code to
>> not compute nfields in binary mode? That seems at best an unnecessary
>> change, and at worst a breakage of the binary path --- did you test it?

> AFAICT it's not used in binary mode at all. But I will double check.

Well, even if it is not used at the moment, it seems potentially of use
in that path. So I'd vote for continuing to set it correctly, rather
than making it deliberately incorrect as this patch is going out of its
way to do.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2010-12-06 18:24:27 Re: WIP patch for parallel pg_dump
Previous Message Tom Lane 2010-12-06 18:15:47 Re: serializable read only deferrable