On Thu, Feb 10, 2011 at 03:25, Stefan Fuhrmann
<stefanfuhrmann_at_alice-dsl.de> wrote:
> Hi there,
>
> Those among you following the commits will have noticed
> that I opened 5 new branches each containing a cherry-
> picked patch set from the performance branch (details below).
>
> These branches can be merged independent of each other and
> an integration branch will be merged to /trunk on the 19th
> or 20th if
>
> * there was no feedback to it at all (= silent acceptance), or
> * it received positive feedback (= +1 on merge), or
> * issues found have been resolved
>
> So, an easy way to prevent a particular branch to be merged
> is a simple "needs more discussion / review" post.
I reviewed some of branches. See my comments below.
> -- Stefan^2.
>
>
> List of integration branches:
> -----------------------------
>
> integrate-string-improvement
> - relatively simple change that mainly speeds up ra_svn clients
>
This branch merges with many conflicts. And after it seems to merge
changes to zlib code...
> integrate-compression-level
> - new, independent feature that allows ra_svn to saturate
> 1Gb network connections and reduces CPU load on the server
- cosmetics: invalid indentation of comment to function
svn_txdelta_to_svndiff2() in file subversion/include/svn_delta.h
- it would be great to have similar option to configure compression
level in mod_dav_svn for high-load servers used in local area
networks. Optionally we can add Apache environment variable to be able
configure compression level based on client IP address.
> integrate-io-improvements
> - changes were already discussed when svn_io_file_read_full2()
> got introduced on the performance branch
Looks fine. I'm +1 to merge.
--
Ivan Zhakov
Received on 2011-02-10 22:16:08 CET