| From: | Sokolov Yura <funny(dot)falcon(at)postgrespro(dot)ru> |
|---|---|
| To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
| Cc: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, pgsql-hackers(at)postgresql(dot)org, Heikki Linnakangas <hlinnaka(at)gmail(dot)com>, pgsql-hackers-owner(at)postgresql(dot)org |
| Subject: | Re: Small improvement to compactify_tuples |
| Date: | 2017-05-17 14:46:19 |
| Message-ID: | 468f371fe05c1de09c4dc101435ef93d@postgrespro.ru |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Alvaro Herrera писал 2017-05-15 20:13:
> As I understand, these patches are logically separate, so putting them
> together in a single file isn't such a great idea. If you don't edit
> the patches further, then you're all set because we already have the
> previously archived patches. Next commitfest starts in a few months
> yet, and if you feel the need to submit corrected versions in the
> meantime, please do submit in separate files. (Some would even argue
> that each should be its own thread, but I don't think that's
> necessary.)
Thank you for explanation.
I'm adding new version of first patch with minor improvement:
- I added detection of a case when all buckets are trivial
(ie 0 or 1 element). In this case no need to sort buckets at all.
--
Sokolov Yura aka funny.falcon
Postgres Professional: https://postgrespro.ru
The Russian Postgres Company
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-Improve-compactify_tuples.patch | text/x-diff | 6.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bossart, Nathan | 2017-05-17 15:06:22 | Re: [Proposal] Allow users to specify multiple tables in VACUUM commands |
| Previous Message | Tom Lane | 2017-05-17 14:23:20 | Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run. |