Re: [PATCH in progress] Pristine text store - pristine_install

On Fri, Feb 18, 2011 at 11:38, Julian Foad <julian.foad_at_wandisco.com> wrote:
> On Fri, 2011-02-18 at 17:20 +0100, Branko Čibej wrote:
>...
>> If I read that patch correctly, you expect to use one sqlite transaction
>> per pristine file? While this may not affect WC performance now, it'll
>> surely show up once other queries are optimized and transactions made a
>> bit less granular.
>
> Yup - I mentioned that earlier today in the other thread "Pristine store
> - spec": I think I'll need to extend so that a buch of pristines can be
> installed at once.  Glad to find you're thinking the same.  But I'll get
> the concurrency guarantees sorted out before extending it.

Let's get the functionality correct, and then worry about the
lifetimes of sqlite transactions.

I mean... how far behind are we? If we're not terribly behind, then I
would suggest publishing 1.7 as-is. Then we can look at a larger body
of work to reduce the sqlite txn's in the 1.8 timeframe. (eg. some
lower-level APIs that auto-bulk changes into a large txn).

Can somebody provide a pointer to some of the latest speed analysis?

Thanks,
-g
Received on 2011-02-19 03:20:35 CET