| 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-09-12 15:49:01 |
| Message-ID: | a9327865736cd63820f009141917063f@postgrespro.ru |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2017-07-21 13:49, Sokolov Yura wrote:
> On 2017-05-17 17:46, Sokolov Yura wrote:
>> 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.
>
> I'm putting rebased version of second patch.
Again rebased version of both patches.
Now second patch applies cleanly independent of first patch.
--
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 |
| 0002-Simplify-PageRepairFragmentation.patch | text/x-diff | 3.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andres Freund | 2017-09-12 15:54:22 | Re: Automatic testing of patches in commit fest |
| Previous Message | Elvis Pranskevichus | 2017-09-12 15:48:28 | Re: [PATCH v1] Add and report the new "in_hot_standby" GUC pseudo-variable. |