Tryton - Issues

 

Issue7119

Title Add cache to check_xml_record
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type performance Components trytond
Assigned To ced Keywords review
Reviews 37991002
View: 37991002

Created on 2018-02-06.15:56:05 by ced, last changed by roundup-bot.

Messages
New changeset 800d673bc90d by C├ędric Krier in branch 'default':
Add cache for check_xml_record
http://hg.tryton.org/trytond/rev/800d673bc90d
review37991002 updated at https://codereview.tryton.org/37991002/#ps40001
review37991002 updated at https://codereview.tryton.org/37991002/#ps20001
review37991002 updated at https://codereview.tryton.org/37991002/#ps1
msg38240 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-02-06.15:56:04
On each modification or deletion, check_xml_record is called and do a search in ModelData. But most of the time, the result is empty because we have a very few number of records created by XML and mainly ir.* stuffs.
But we can not cache every query as the possibilities are infinite. So the idea is to limit by model when to do the search. There are only a few models that have records created by XML so we can keep them in the cache and skip the search for all model not in the list.
History
Date User Action Args
2018-02-14 19:41:33roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg38377
2018-02-08 20:26:27reviewbotsetmessages: + msg38281
2018-02-06 16:55:17reviewbotsetmessages: + msg38244
2018-02-06 16:26:35reviewbotsetnosy: + reviewbot
messages: + msg38241
2018-02-06 16:10:51cedsetstatus: in-progress -> testing
reviews: 37991002
keyword: + review
2018-02-06 15:56:05cedcreate

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