Re: Small improvement to compactify_tuples

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

In response to

Responses

Browse pgsql-hackers by date

  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.