[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Performance branch review: single revision changes

From: Stefan Fuhrmann <stefanfuhrmann_at_alice-dsl.de>
Date: Mon, 20 Sep 2010 11:52:37 +0200

Hyrum K. Wright wrote:
> On Mon, Sep 20, 2010 at 7:47 AM, Bert Huijben <bert_at_qqmail.nl> wrote:
>
>>
>>> -----Original Message-----
>>> From: Stefan Fuhrmann [mailto:stefanfuhrmann_at_alice-dsl.de]
>>> Sent: zondag 19 september 2010 21:59
>>> To: Subversion Development
>>> Subject: Performance branch review: single revision changes
>>>
>>> Hi there,
>>>
>>> as per public request, this is a list of smaller changes made on the
>>> performance branch. Each revision can be reviewed (hopefully)
>>> without further context and be merged into trunk independently.
>>>
>>> * r982335
>>> Limit the amount of unused memory kept in apr_pools
>>> to the same amount as everywhere else, e.g. SVN.exe main().
>>> Justification:
>>> (1) uniform handling of memory pools
>>> (2) Without this change, apr pools will fragment indefinitely (?)
>>> for strings (e.g. fulltexts) and other chunks larger than 80kB.
>>> This already happens on trunk if memcached usage has been
>>> enabled.
>>>
>> (not related to the content, specific patches)
>> This mail looks like how we handle branches/1.Y.x/STATUS
>>
>> Maybe you should check it in to your branch to allow further updates (and
>> review) there :)
>>
>
> A good strategy.
>
> And thanks for sending this, Stefan, it will help this work get onto
> trunk quicker.
>
> -Hyrum
>
Done in r998858.
I was hesitant to do it in the first place because the
review / merge direction is reversed in comparison
to the 1.y.x branches.

-- Stefan^2.
Received on 2010-09-20 11:55:23 CEST

This is an archived mail posted to the Subversion Dev mailing list.