Issue 10491

Title
When a function field use another function field the latter is computed twice
Priority
feature
Status
resolved
Nosy list
nicoe, reviewbot, roundup-bot
Assigned to
nicoe
Keywords
review

Created on 2021-06-07.09:35:25 by nicoe, last changed 3 months ago by roundup-bot.

Messages

New changeset c3b707822256 by Nicolas √Čvrard in branch 'default':
Use global cache for Function fields in readonly transactions
https://hg.tryton.org/tryton-env/rev/c3b707822256
New changeset 05235e67b280 by Nicolas √Čvrard in branch 'default':
Use global cache for Function fields in readonly transactions
https://hg.tryton.org/trytond/rev/05235e67b280
Author: [hidden] (nicoe) Tryton committer
Date: 2021-06-07.09:35:24

It happens sometimes that a function fields depends on another function fields but that it's not manageable to use the names trick to compute them all at once. In order to have better performances it would be useful to prevent the recomputation of the "base" function fields when possible.

The attached patch does so by adding the concept of reading priority on function fields and by sharing the dictionary used for the local_cache when computing them.

History
Date User Action Args
2021-10-11 11:52:55roundup-botsetmessages: + msg70854
2021-10-11 11:52:51roundup-botsetmessages: + msg70853
nosy: + roundup-bot
status: in-progress -> resolved
2021-10-08 11:11:20reviewbotsetmessages: + msg70762
2021-09-21 15:00:46reviewbotsetmessages: + msg70244
2021-09-01 19:11:57reviewbotsetmessages: + msg69768
2021-07-16 19:28:53reviewbotsetmessages: + msg68918
2021-06-10 18:36:00reviewbotsetmessages: + msg68185
2021-06-09 11:29:49reviewbotsetmessages: + msg68165
2021-06-07 10:02:57reviewbotsetmessages: + msg68123
nosy: + reviewbot
2021-06-07 09:35:25nicoecreate

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