Re: Early warning: Major log cache code update

On 10/5/07, Stefan.Fuhrmann@etas.de <Stefan.Fuhrmann@etas.de> wrote:

> under /branches/LogCacheEnhancement, I developed a number of
> Improvements that I would like to merge into /trunk. My plan
> is to do that some time between 2007-10-06 00:00 and 12:00 UTC.
>
> As a "vital" part of the code gets touched (see below), there
> may be blocking errors in it. Please speak up if you want the
> merge to be deferred.
>
> Changes done so far:
>
> * Log cache files are ~20% smaller now
> * Log cache can be saved and loaded ~30% faster
> * Log caches now support merge tracking information and
> user-defined revision properties

how did you get the merge tracking information if (as you mention
below) you don't use svn_client_log4() yet? Or do I misunderstand
something here?

> * Log caches now support partial information (e.g. merge info
> not available for a given revision)

What happens if merge info is not available for a revision? Does the
cache re-fetch it from the repository?

> * Log caches can now be updated, including partial update
> (e.g. only the author of a given revision)
>
> -> Existing log caches can not be used and will be overwritten
> automatically.

That's not a problem.

> * Changing author or comment in the log dialog will update
> the log cache as well.
>
> * Repository properties (root URL, UUID, one path-head-pair)
> are now cached as well
> * Log dialog and revision graph use the repository property
> cache to elimilate most server-roundtrips.
> * Head revision info is cached for the last queried path only
> and is considered valid for 5 minutes (registry setting
> HeadCacheAgeLimit:DWORD in seconds).
>
> * Minor fixes.
>
> To do before merging into /trunk:
>
> * Refresh in log dialog shall update the head revision
> (e.g. "any new revisions?") as well as update authors and
> comments in the revisions shown.

I think to update authors/comments, the users can use the 'clear log
cache' in the settings dialog. A refresh in the log dialog should not
invalidate the cache IMHO.

Stefan

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org