Re: svn commit: r1643189 - in /subversion/branches/1.7.x: ./ subversion/ subversion/include/svn_repos.h subversion/libsvn_repos/load-fs-vtable.c subversion/tests/libsvn_repos/repos-test.c subversion/t

2014-12-08 Thread Ben Reser
With careful effort of stefan2 and myself we've reverted this. It was mistakenly merged onto the 1.7.x branch not the 1.7.x-r1643074 branch. Not only that but the merge it says it's doing is not what it did, it was actually a merge of r1643119 on 1.8.x-r1643074. I found this only because the mer

[l10n] Translation status report for trunk r1643983

2014-12-08 Thread Subversion Translation Status
Translation status report for trunk@r1643983 lang trans untrans fuzzy obs -- de2871 2 3 489 +~o es2263 610 827 527 ++U~~~ fr2567 306

Re: On pool / memory usage debugging

2014-12-08 Thread Philip Martin
Philip Martin writes: > My pool-debug build uses my own httpd build from 2.2.x@1562432 while my > normal build used the system httpd, Debian's 2.2.22-13+deb7u3. It looks > like some change to mod_dav has added an extra walk over the copy source > in the pool-debug build and the absence of this w

Re: On pool / memory usage debugging

2014-12-08 Thread Philip Martin
Philip Martin writes: > I think this extra walk is pointless most of the time and is just making > our nominally O(1) copy operation slower. Locks on the copy source do > not prevent a copy. Oops! I meant to write "Locks on the copy source held by somebody else do not prevent a copy". -- Phil

Re: On pool / memory usage debugging

2014-12-08 Thread Philip Martin
Philip Martin writes: > I'm still trying to determine why this > extra walk is present, perhaps the new version is doing an unnecessary > walk or perhaps the old version has an unfixed bug. Debian's httpd doesn't have r1497441 which introduced the extra walk for copies, apparently to fix PR 5461

Re: On pool / memory usage debugging

2014-12-08 Thread Philip Martin
Philip Martin writes: > The problem is easy to reproduce with pool debugging enabled and the > patch does reduce memory use, but with a normal build I don't see the > excessive memory in the first place. My pool-debug build uses my own httpd build from 2.2.x@1562432 while my normal build used th

Re: On pool / memory usage debugging

2014-12-08 Thread Stefan Sperling
On Mon, Dec 08, 2014 at 08:30:31PM +0100, Stefan Fuhrmann wrote: > Hm. 381 MB are massive, then. Maybe I can find reproduce it > and help tracking it down with a modified APR. Yes, we should be able to manage with much less. Though perhaps what's left is in in mod_dav rather than mod_dav_svn. > >

Re: On pool / memory usage debugging

2014-12-08 Thread Stefan Sperling
On Mon, Dec 08, 2014 at 07:19:57PM +, Philip Martin wrote: > I have been looking at the proposed 1.8/1.7 backport for issue 4531. > The problem is easy to reproduce with pool debugging enabled and the > patch does reduce memory use, but with a normal build I don't see the > excessive memory in

Re: On pool / memory usage debugging

2014-12-08 Thread Stefan Fuhrmann
On Mon, Dec 8, 2014 at 7:46 PM, Stefan Sperling wrote: > On Mon, Dec 08, 2014 at 06:42:38PM +0100, Stefan Fuhrmann wrote: > > IOW, pool debugging is nice for tracing allocations but if you > > want to measure memory consumption on the OS side, turn > > pool debugging off. > > All measurements I m

Re: On pool / memory usage debugging

2014-12-08 Thread Philip Martin
Stefan Sperling writes: > Which option are you referring? The SVNInMemoryCacheSize option? > The doc for that option says "0 deactivates the cache". Is this an error? http://subversion.apache.org/docs/release-notes/1.7.html#data-caches "Please note that a cache size of 0 will deactivate the new

Re: On pool / memory usage debugging

2014-12-08 Thread Philip Martin
Stefan Fuhrmann writes: > This post has been prompted by issue 4531 and r1643834 > (http://subversion.tigris.org/issues/show_bug.cgi?id=4531 > http://svn.apache.org/viewvc?view=revision&revision=r1643834). > IOW, pool debugging is nice for tracing allocations but if you > want to measure memory

Re: On pool / memory usage debugging

2014-12-08 Thread Stefan Sperling
On Mon, Dec 08, 2014 at 06:42:38PM +0100, Stefan Fuhrmann wrote: > IOW, pool debugging is nice for tracing allocations but if you > want to measure memory consumption on the OS side, turn > pool debugging off. All measurements I mentioned in the issue were done with pool debugging disabled. Measur

On pool / memory usage debugging

2014-12-08 Thread Stefan Fuhrmann
This post has been prompted by issue 4531 and r1643834 (http://subversion.tigris.org/issues/show_bug.cgi?id=4531 http://svn.apache.org/viewvc?view=revision&revision=r1643834). We had a somewhat similar issue with log -v & friends in summer this year. Now, I dug into the APR code and this what I fo

Re: Removing leftover backport branches

2014-12-08 Thread Julian Foad
Stefan Fuhrmann wrote: > Julian Foad wrote: >> So now we need to: >> >>  * undo the merge of the 1.8.x-r1611379 branch > > +1. r1643849. >>  * re-nominate my original nomination. > > +1. r1643849 and r1643850. I down-graded all our votes (including my own) from +1 to +0 because whatever we d

Re: Removing leftover backport branches

2014-12-08 Thread Stefan Fuhrmann
On Mon, Dec 8, 2014 at 5:07 PM, Julian Foad wrote: > Stefan Fuhrmann wrote: > > Looking through ^/subversion/branches, I found that there are many > > backport branches that are not mentioned in any STATUS file. > [...] > > > 1.8.x > > Julian: 1.8.x-r1619380 (not modified) > > Ugh. Thanks for rep

Re: Removing leftover backport branches

2014-12-08 Thread Julian Foad
Stefan Fuhrmann wrote: > Looking through ^/subversion/branches, I found that there are many > backport branches that are not mentioned in any STATUS file. [...] > 1.8.x > Julian: 1.8.x-r1619380 (not modified) Ugh. Thanks for reporting this discrepancy. There is a mess here. First, it IS modified

Re: svn commit: r1643793 - /subversion/trunk/autogen.sh

2014-12-08 Thread Philip Martin
Branko Čibej writes: > On 08.12.2014 13:03, phi...@apache.org wrote: >> Author: philip >> Date: Mon Dec 8 12:03:23 2014 >> New Revision: 1643793 >> >> URL: http://svn.apache.org/r1643793 >> Log: >> * autogen.sh: Unset CDPATH. >> >> Modified: >> subversion/trunk/autogen.sh >> >> Modified: sub

Re: svn commit: r1643793 - /subversion/trunk/autogen.sh

2014-12-08 Thread Branko Čibej
On 08.12.2014 13:03, phi...@apache.org wrote: > Author: philip > Date: Mon Dec 8 12:03:23 2014 > New Revision: 1643793 > > URL: http://svn.apache.org/r1643793 > Log: > * autogen.sh: Unset CDPATH. > > Modified: > subversion/trunk/autogen.sh > > Modified: subversion/trunk/autogen.sh > URL: > ht

Re: The assert subroutine failed ( file subversion/libsvn_delta/text_delta.c )

2014-12-08 Thread Stefan Fuhrmann
On Mon, Dec 8, 2014 at 10:44 AM, Naveen Mc wrote: > Hi, > I am facing the following issue when i try to sync master and mirror svn > repositories. Any idea how do i fix this.Is it svn 1.5 known bug? Please > help. > > OS - AIX/Linux > SVN Version - 1.5 > ERROR: >- Failed to sync apd repos [ ]

The assert subroutine failed ( file subversion/libsvn_delta/text_delta.c )

2014-12-08 Thread Naveen Mc
Hi, I am facing the following issue when i try to sync master and mirror svn repositories. Any idea how do i fix this.Is it svn 1.5 known bug? Please help. OS - AIX/Linux SVN Version - 1.5 ERROR: - Failed to sync apd repos [ ]. The assert subroutine failed: window->sview_len == 0 || (window->sv