Re: Status of wc-propcaching branch

On Mon, 28 Nov 2005, Greg Hudson wrote:

> 3. Declare once per entries file which properties we are caching the
> presence of, and later declare which of those is present.
>

To me, this means adding another field to the svn_wc_entry_t struct, a
pointer to a string of cachable props for this entry. This is inherited
form the THIS_DIR entry. This is quite cheap; a pointer per entry and a
string per directory. We could have it in the access baton, but since
entries aren't always read when opening a baton (depth 0), this would
complicate other code.

> 4. Declare the presence of all properties.
>
This is the simple and ellegant solution.

> I'd advocate choosing between (3) and (4) based on how complex (3) comes
> out to be.
>
I agree that 3 and 4 are the best choices. I'll give 3 a shot and if it
doesn't turn out to be too complex, I'll use it. Else, I vote for 4. I
think 4 is only a problem in more or less pathological cases, but OTOH we
don't need to design more non-scalability than we need to:-)

Thanks,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Nov 28 23:39:04 2005