Tryton - Issues



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.

New changeset 800d673bc90d by C├ędric Krier in branch 'default':
Add cache for check_xml_record
review37991002 updated at
review37991002 updated at
review37991002 updated at
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.
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)