On 9/25/06, Hanley, James <jhanley@redcom.com> wrote:
> If, as you mention, there could be a tweak to the the blame to provide
> information for removed lines of text, then perhaps just including the
> lines removed and indicating the version they were removed in would
> suffice -- although I tried as a test an svn blame on one of our heavily
> versioned files (com/something/res/lang_toks.properties) which every one
> is adding to and it spun the apache server for a good 5 minutes before
> spitting back a svn blame... not sure if adding this tweak would worsen
> that time by an order of magnitude but I could definitely could see it
> might with adding an svn grep depending on the implimentation.
The only difference would be that the deltas needed to assemble the
revisions of the file are sent over the wire and the search through
them would be run on the client instead of the server. Other than
that the operation is identical. You're still reconstituting all the
revisions of the file to search through, it's going to be expensive no
matter what.
Of course the main advantage of using the same underlying
implementation as blame (the svn_ra_get_file_revs call, for the
curious) is that it exists today, so you don't have to wait until a
server that supports your hypothetical new operation shows up, you
could implement the functionality right now.
-garrett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Sep 25 23:47:06 2006