Tryton - Issues

 

Issue7359

Title Improve loading strategy
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type performance Components sao, tryton
Assigned To ced Keywords review
Reviews 46361002,47351002
View: 46361002, 47351002

Created on 2018-04-16.17:42:24 by ced, last changed by roundup-bot.

Messages
New changeset 6a3d7715f9c7 by Cédric Krier in branch 'default':
Add views of the field in the loading strategy
https://hg.tryton.org/sao/rev/6a3d7715f9c7
New changeset c043d5706cf3 by Cédric Krier in branch 'default':
Add views of the field in the loading strategy
https://hg.tryton.org/tryton/rev/c043d5706cf3
review47351002 updated at https://codereview.tryton.org/47351002/#ps100001
review47351002 updated at https://codereview.tryton.org/47351002/#ps80001
review46361002 updated at https://codereview.tryton.org/46361002/#ps100001
review47351002 updated at https://codereview.tryton.org/47351002/#ps60001
review46361002 updated at https://codereview.tryton.org/46361002/#ps80001
review47351002 updated at https://codereview.tryton.org/47351002/#ps40001
review46361002 updated at https://codereview.tryton.org/46361002/#ps60001
review47351002 updated at https://codereview.tryton.org/47351002/#ps20001
review46361002 updated at https://codereview.tryton.org/46361002/#ps40001
review46361002 updated at https://codereview.tryton.org/46361002/#ps20001
review47351002 updated at https://codereview.tryton.org/47351002/#ps1
review46361002 updated at https://codereview.tryton.org/46361002/#ps1
msg40162 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-04-16.17:42:23
For now, the strategy is based on the loading keyword 'eager' or 'lazy'.
On list, it loads all eager fields and on form it loads all fields.
But if the screen has many views (especially form) which show different fields, the client will often load too many fields.
We could improve the behavior by tying the field to the views where it is displayed. So when selecting other fields to load we could limit the selection to other fields which share a view.
Also when a view load fields, it should try first the fields displayed the less (by chance it will be only display on the view) to reduce the number of field selected.
History
Date User Action Args
2018-06-20 11:22:22roundup-botsetmessages: + msg41489
2018-06-20 11:21:35roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg41488
2018-06-19 15:28:15reviewbotsetmessages: + msg41474
2018-06-18 23:34:32reviewbotsetmessages: + msg41463
2018-06-18 23:34:15reviewbotsetmessages: + msg41462
2018-06-11 23:32:29reviewbotsetmessages: + msg41363
2018-06-11 23:32:13reviewbotsetmessages: + msg41362
2018-05-08 14:13:31reviewbotsetmessages: + msg40570
2018-05-08 14:13:13reviewbotsetmessages: + msg40569
2018-04-30 15:58:46reviewbotsetmessages: + msg40476

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