| 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: | Whole Thread | Raw Message | 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 |