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

Re: Subversion performance (issue #1429 et al)

From: <cmpilato_at_collab.net>
Date: 2003-08-17 18:56:39 CEST

"Sander Striker" <striker@apache.org> writes:

> > svn_fs_dir_entries() uses the not-cheap open_path() routine to walk
> > the DAG down to the directory, then reads the dirents list from the
> > reps/strings data, then opens each dirent directly and asks the
> > kind question, then returns the dirents with their kinds. Note
> > that no open_path() is needed here because we already have the
> > node-rev-ids of the entries. In other words, this is *far* cheaper
> > to do inside the API than the equivalent work outside the API.
>
> Can't we cache those node-rev-ids? So that if we have already called
> open_path() we don't need to do it again? I am quite likely missing
> something here...

We used to have a mechanism for doing this (a third svn_fs_root_t type
called an "id_root"), but the concept was sorta broken, and exposed
filesystem details that are best not exposed.

> Good. That will make a big difference for performance.
>
> Other functions to look at in this light:
> - svn_fs_node_created_rev
> - svn_fs_file_md5_checksum
> - svn_fs_file_contents
> - svn_fs_node_proplist
> - svn_fs_get_file_delta_stream

Gotcha.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Aug 17 18:59:46 2003

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.