Re: [Maposmatic-dev] [PATCH maposmatic 1/3] Cornerstones for a new MapOSMatic daemon

[David MENTRE]

Hello Maxime,

2010/2/5 Maxime Petazzoni <address@hidden>:
> This is the first step of the rewrite of the MapOSMatic rendering
> daemon. The long-term objective is to get a more flexible rendering
> daemon that would support the rendering of several jobs in parallel, and
> with an overall more Python-y and maintainable code.

I have finally (quickly) looked at your code. No special opinion about
it. I'm not a huge fan of threading and all those parallel things (to
say the least). For me it can quickly become a big source of issues
and their advantages (if any) should be carefully balanced with added
complexity. I would have preferred the cleanup being done serially as
before and not in a separate thread. But as you have written it that
way...

In short: go ahead, you'll debug it if it breaks. ;-)

Best regards,
d.