Community mailing list archives

Re: Odoo Performance and concurrency locks

OpenERP Master
- 09/14/2015 10:57:59
You know the original reason the select for update nowait was introduced was because we were testing v6 GTK client and noticed while the MRP scheduler was running, no one could login to the system. The scheduler effectively locked everyone out of the system. I would like us to ensure that any modification we introduce does not regress that problem.

I don't understand why when this statement is executed the oldest thread is killed first.

A. Admin user running batch job
B. some time later admin user opens new tab or logs in again,

Result of action B Kills action A and B succeeds. It seems a more appropriate action would be that B fails, login succeeds but login time is not updated. This appears to be the design intent, however it does not work correctly.

On Mon, Sep 14, 2015 at 3:22 AM, Alexandre Fayolle <> wrote:
On 14/09/2015 09:47, Nhomar Hernández wrote:
> 2015-09-14 1:33 GMT-05:00 Alexandre Fayolle
> <
> <>>:
>     Install base_concurrency ( this will
>     alleviate lots of issues.
> Alexandre, did you know if this situation was solved (using a similar
> algorithm) in v9?
> It should be cool if adfter the pre-release we can fix this before go to
> stable don't you think?
> Did somebody made the PR?
> @odony, don't you think it is an important issue which should be solved
> (on this way or another)?

I know I did not make a PR and I'm not aware of any colleague making
one. The principle of the fix are easy to understand, and there is no
objection from Camptocamp (or the OCA) to Odoo taking the code from that
module and applying it to the core (provided authorship is preserved),
or to roll their own. We chose to make this a separate module rather
than an OCB improvement so that people using the official version could
still benefit from the fix.

No idea if this can be merged in v9 at this point in time, though.

Alexandre Fayolle
Chef de Projet
Tel : +33 4 58 48 20 30

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex

Post to: