Issue 10344

Title
ActivePeriodMixin use on General Ledger reports result in a huge performance penalty
Priority
feature
Status
in-progress
Nosy list
albertca, ced, nicoe, pokoli, reviewbot
Assigned to
nicoe
Keywords
review

Created on 2021-04-25.16:45:28 by nicoe, last changed 1 week ago by reviewbot.

Messages

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-04-25.17:48:44

Indeed I think search_active could be reimplemented as domain_active to use directly the column of the table.

Author: [hidden] (nicoe) Tryton committer
Date: 2021-04-25.16:45:28

Because of the structure of the active field search (which basically makes an id IN SELECT id FROM general_ledger WHERE some_date_conditions) it results in an almost doubling of the time spent to execute the query.

We should probably reimplement search_active for the models inheriting from GeneralLedger.

History
Date User Action Args
2021-04-27 12:23:38reviewbotsetmessages: + msg67000
nosy: + reviewbot
2021-04-27 12:01:46nicoesetassignedto: nicoe
2021-04-27 12:01:40nicoesetkeyword: + review
reviews: 351841002
status: chatting -> in-progress
2021-04-26 09:56:47pokolisetnosy: + pokoli
2021-04-25 22:22:02albertcasetnosy: + albertca
2021-04-25 17:48:50cedsetcomponent: + account
2021-04-25 17:48:44cedsetmessages: + msg66942
nosy: + ced
status: unread -> chatting
2021-04-25 16:45:28nicoecreate

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