Re: [Monotone-devel] Performance Issues With Version 0.40

[Zack Weinberg]

On Thu, Sep 4, 2008 at 9:28 AM, Timothy Brownawell <address@hidden> wrote:
> Running lots (500) of get_content_changed commands thru automate stdio,
> it seems to be around 1 1/2 times slower here. There's about a 46%
> speedup from hand-coding decode_hexenc (instead of using Botan),

We should probably just go ahead and do that, but I'd like to know
what things we still have to decode_hexenc - the 0.40 database changes
ought to have meant we need to do that way less often.

> about
> 38% from making the db variable in the get_content_changed command
> static (I think most of this is because initializing the db each time
> includes verifying the schema each time),

Oh, ick.

A cleaner approach would probably be to cache the sqlite handle in the
app_state.  I didn't want to go there initially because I was planning
on doing the ro/rw_database distinction that might need to close and
reopen sqlite handles to get the effect we want, but since I don't
have time to do that anytime soon, perhaps we should do the caching
anyway.

zw