On Thu, Feb 10, 2011 at 03:25, Stefan Fuhrmann <stefanfuhrm...@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
