From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Heikki Linnakangas <heikki(at)enterprisedb(dot)com> |
Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: CopyReadAttributesCSV optimization |
Date: | 2008-03-08 00:16:11 |
Message-ID: | 47D1DACB.8020108@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Heikki Linnakangas wrote:
> Here's a patch to speed up CopyReadAttributesCSV. On the test case
> I've been playing with, loading the TPC-H partsupp table, about 20%
> CopyReadAttributesCSV (inlined into DoCopy, DoCopy itself is
> insignificant):
>
>
[snip]
>
> The trick is to split the loop in CopyReadAttributesCSV into two
> parts, inside quotes, and outside quotes, saving some instructions in
> both parts.
>
> Your mileage may vary, but I'm quite happy with this. I haven't tested
> it much yet, but I wouldn't expect it to be a loss in any interesting
> scenario. The code also doesn't look much worse after the patch,
> perhaps even better.
>
>
This looks sane enough, and worked for me in testing, so I'm going to
apply it shortly. I'll probably add a comment or two about how the loops
interact.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-03-08 01:13:11 | Re: Remove FATAL from pg_lzdecompress |
Previous Message | Tom Lane | 2008-03-07 23:23:12 | Re: Proposed patch to change TOAST compression strategy |