Re: Database threading + WorkerThread, v0

[Ruben Vermeersch <ruben Lambda1 be>]

For the interested, I've posted this one in bugzilla. When testing,
please use the patch in bugzilla, as it's a slightly improved version
compared to this one.

It's over here: http://bugzilla.gnome.org/show_bug.cgi?id=327107

Kind regards,
   Ruben

On Fri, 2006-04-07 at 19:35 +0200, Ruben Vermeersch wrote:
> Hi all, 
> 
> I've just completed a rather large patch, which rewrites the database
> layer almost completely and adds a background thread to do long running
> & heavy tasks.
> 
> I started this because I was annoyed with the way XMP metadata is
> written to files, it blocks the UI. Therefor I added a background
> thread, which can be used to do all sorts of background tasks (I'm
> thinking of stuff like automatically syncing tags with flickr).
> 
> In the process, I had to make the database code thread-aware, as sqlite
> doesn't handle threads very well. With the help of Abock, this has been
> implemented, largely based on banshee's database code (which has been
> known to work ;-)).
> 
> Anyway, I'm going to post this in the bugzilla in a couple of days, but
> I'd like to ask for some feedback/reviews here first.
> 
> Here's how to test it:
>  * Start f-spot without the patch, enable XMP metadata writing, add a
> tag to 70+ pictures. You'll notice the UI hangs quite a long time (with
> loads of console output).
>  * Now try the same with the patch applied. Tagging should be a lot
> faster. Metadata is written in the background. As a side-effect, f-spot
> is now threading-capable, so we can think about doing more things
> outside of the GTK mainloop (thus less blocking of the UI).
> 
> 
> Any feedback would be greatly appreciated!
> 
> Kind regards,
>    Ruben 


--
Ruben Vermeersch (rubenv)
http://www.Lambda1.be/