"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