What is the status of issue #2151?

[Kazutoshi Satoda <k_...@f2.dion.ne.jp>]

I have encountered issue #2151 using client/server 1.3.0.

There is a patch from Jean-Marc Godbout for this issue in mail
archives(http://svn.haxx.se/dev/archive-2005-09/0009.shtml).
But the patch is not committed. So I downloaded the patch and tried to
apply this patch to trunk. Then conflicts was occurred in
libsvn_ra_dav/fetch.c, ra_dav.h. These conflicts are with r17067
that speeded up non-verbose 'svn list'.

The patch and r17067 is speeding up 'svn list' by avoiding to get
all props at svn_ra_dav__get_dir(). The patch does that by adding new
property 'deadprop_count'. r17067 did that by new argument
'dirent_fields', but did not address issue #2151.

So what is the status of issue #2151 now? Was it resolved by r17067?
If the patch is still effective, is there a chance to get improved
'svn list' in 1.3.x?

Best regards.
--
k_satoda


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: What is the status of issue #2151?

[Kazutoshi Satoda <k_...@f2.dion.ne.jp>]

Garrett Rooney wrote:
> > So what is the status of issue #2151 now? Was it resolved by r17067?
> > If the patch is still effective, is there a chance to get improved
> > 'svn list' in 1.3.x?
> 
> r17067 sped things up, but only for non-verbose list.  The solution
> proposed in issue 2151 would result in a smaller speedup, but would
> work for verbose list as well as non-verbose list.

Thank you for your reply.

Then, issue 2151 seems to still remain even in trunk.

I will work to solve this problem. But I can't do that immediately.
I'm now setting up build environment on my Windows laptop. I will
come again when I got something more concrete(such as a patch, or
useful test results).

Someone more expert in ra_dav/mod_dav_svn is still welcome. Don't
mind me, because this work itself is a fun for me(a novice in these
area).

--
k_satoda

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: What is the status of issue #2151?

[Garrett Rooney <ro...@electricjellyfish.net>]

On 1/29/06, Kazutoshi Satoda <k_...@f2.dion.ne.jp> wrote:
> I have encountered issue #2151 using client/server 1.3.0.
>
> There is a patch from Jean-Marc Godbout for this issue in mail
> archives(http://svn.haxx.se/dev/archive-2005-09/0009.shtml).
> But the patch is not committed. So I downloaded the patch and tried to
> apply this patch to trunk. Then conflicts was occurred in
> libsvn_ra_dav/fetch.c, ra_dav.h. These conflicts are with r17067
> that speeded up non-verbose 'svn list'.
>
> The patch and r17067 is speeding up 'svn list' by avoiding to get
> all props at svn_ra_dav__get_dir(). The patch does that by adding new
> property 'deadprop_count'. r17067 did that by new argument
> 'dirent_fields', but did not address issue #2151.
>
> So what is the status of issue #2151 now? Was it resolved by r17067?
> If the patch is still effective, is there a chance to get improved
> 'svn list' in 1.3.x?

r17067 sped things up, but only for non-verbose list.  The solution
proposed in issue 2151 would result in a smaller speedup, but would
work for verbose list as well as non-verbose list.  It's simply
awaiting someone bringing it up to date with the recent changes and
doing the necessary work to get it committed (i.e. proving that it
improves things measurably without introducing any problems).  Don't
know when it'll make it in, since at this point I don't know of anyone
actively working on it.

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org