On 12/06/2010 01:23 PM, Tom Lane wrote:
Andrew Dunstan<and...@dunslane.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.
Ok.
cheers
andrew
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers