On Thu, Aug 12, 2010 at 4:25 PM, <stef...@apache.org> wrote: > Author: stefan2 > Date: Thu Aug 12 21:25:11 2010 > New Revision: 984984 > > URL: http://svn.apache.org/viewvc?rev=984984&view=rev > Log: > Eliminate redundant revprop lookups: Exports / checkouts often > contain multiple nodes from the same revision. Therefore, we > cache essential revision info in the report baton for as long as > the report is running. > > As a neat side-effect, this will also fix inconsistencies created by > changing revprops (in a parallel request) while a report is running.
A nice change. In reviewing it for merge to trunk, I noticed that cached information is never evicted. Do we assume that the number of revisions touched won't be significant, so we don't need to worry about the size of the cache (and hence memory usage) blowing up? -Hyrum
