Tryton - Issues

 

Issue7939

Title Speedup depends computation
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type performance Components trytond
Assigned To ced Keywords review
Reviews 64521002
View: 64521002

Created on 2018-12-21.15:26:44 by ced, last changed by roundup-bot.

Messages
New changeset 3b212f998ec2 by Cédric Krier in branch 'default':
Compute depends on-demand instead of for all callable
https://hg.tryton.org/tryton-env/rev/3b212f998ec2
New changeset d387b2d058df by Cédric Krier in branch 'default':
Compute depends on-demand instead of for all callable
https://hg.tryton.org/trytond/rev/d387b2d058df
msg46048 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2019-01-18.00:07:14
I measured about 10% faster on startup
review64521002 updated at https://codereview.tryton.org/64521002/#ps1
msg45430 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-12-21.15:26:43
With issue5191, ModelView.__setup__ computes the depends for all callables.
On large setup (with a lot of Models, extensions and methods) this can be slow if most of the callables are not decorated with depends.
We could switch to an on-demand computation as there are not performance boost in computing all in the same loop.
History
Date User Action Args
2019-01-18 00:36:06roundup-botsetmessages: + msg46051
2019-01-18 00:26:09roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg46049
2019-01-18 00:07:14cedsetmessages: + msg46048
2018-12-21 15:40:47reviewbotsetnosy: + reviewbot
messages: + msg45431
2018-12-21 15:28:59cedsetstatus: in-progress -> testing
reviews: 64521002
keyword: + review
2018-12-21 15:26:44cedcreate

Showing 10 items. Show all history (warning: this could be VERY long)