From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
---|---|
To: | Marko Tiikkaja <marko(at)joh(dot)to> |
Cc: | Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: COPY (INSERT/UPDATE/DELETE .. RETURNING ..) |
Date: | 2015-11-26 05:55:17 |
Message-ID: | CAB7nPqTAuKRd1GhjRVFdOtfG_CRz0M4c6L90vOMTcMnFY1NEqQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Nov 22, 2015 at 9:54 AM, Marko Tiikkaja wrote:
> Attached the test case I used; removing a batch of old rows from a table
> through an index. Best out of three runs, I get 13.1 seconds versus 15.0
> seconds, which should amount to an improvement of around 12.7%.
Thanks for the test case. I haven't seen that much, up to 5% on my
laptop still that's not bad thinking that it saves one execution step.
> Also attached a v3 of the patch which adds an Assert on top of the test case
> changes suggested by you.
Sorry for the late reply. I have been playing a bit more with this
patch and I am withdrawing my comments regarding the use of SelectStmt
here. I have looked at the patch again, noticing typos as well as
comments not updated, particularly for psql's \copy, resulting in the
attached.
Patch is switched to "ready for committer".
--
Michael
Attachment | Content-Type | Size |
---|---|---|
20151126_copydml_v4.patch | application/x-patch | 17.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2015-11-26 06:25:06 | Re: [PATCH] Reload SSL certificates on SIGHUP |
Previous Message | Kyotaro HORIGUCHI | 2015-11-26 05:45:12 | Re: Making tab-complete.c easier to maintain |