Issue 10026

Title
Instantiate only requested field attribute
Priority
feature
Status
resolved
Nosy list
ced, reviewbot, roundup-bot
Assigned to
ced
Keywords
review

Created on 2021-01-24.01:48:36 by ced, last changed 2 months ago by roundup-bot.

Messages

New changeset e0cd23dd07e4 by Cédric Krier in branch 'default':
Do not instantiate other fields loaded eagerly
https://hg.tryton.org/tryton-env/rev/e0cd23dd07e4
New changeset 42deab7e7838 by Cédric Krier in branch 'default':
Do not instantiate other fields loaded eagerly
https://hg.tryton.org/trytond/rev/42deab7e7838
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-01-24.01:48:36

It is not needed to instantiate other fields that were eagerly loaded because we do not know if they will be needed. But as there raw value is stored in the global cache, when they will be requested they could be instantiated with only the value in the cache.
Also there is no benefit to instantiate other fields with the requested one as they do not share anything that could improve the loading. (Indeed if two fields have the same Model, currently their ids will be interleaved but it is not really better than having two lists).

History
Date User Action Args
2021-02-13 11:16:07roundup-botsetmessages: + msg64534
2021-02-13 11:16:03roundup-botsetmessages: + msg64533
nosy: + roundup-bot
status: testing -> resolved
2021-01-24 01:53:27reviewbotsetmessages: + msg64063
nosy: + reviewbot
2021-01-24 01:51:48cedsetkeyword: + review
reviews: 341251002
status: in-progress -> testing
2021-01-24 01:48:36cedcreate

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