Tryton - Issues

 

Issue7132

Title Improve reconciliation performance
Priority feature Status resolved
Superseder Nosy List ced, reviewbot, roundup-bot
Type performance Components account, account_statement
Assigned To ced Keywords review
Reviews 41051002,41061002
View: 41051002, 41061002

Created on 2018-02-13.12:40:55 by ced, last changed by roundup-bot.

Messages
New changeset f00bbd0dea73 by Cédric Krier in branch 'default':
Use new API for Line.reconcile
http://hg.tryton.org/modules/account_statement/rev/f00bbd0dea73
New changeset 394132ec89d2 by Cédric Krier in branch 'default':
Allow to reconcile many group of lines with Line.reconcile
http://hg.tryton.org/modules/account/rev/394132ec89d2
review41061002 updated at https://codereview.tryton.org/41061002/#ps1
review41051002 updated at https://codereview.tryton.org/41051002/#ps1
msg38333 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-02-13.12:40:54
The creation of reconciliation trigger the processing of the invoices so for performance reason it is important to create reconciliation records grouped.
But the Line.reconcile method, which is the standard way to create reconciliation, create one reconciliation at a time.
So I propose to improve the API in the best backward compatible way to allow to reconcile a list of lines and thus create all the reconciliation at once.
The account statement could benefit from this design to reconcile all the lines linked to an invoice at once.
History
Date User Action Args
2018-04-04 22:29:36cedlinkissue5276 superseder
2018-03-26 11:11:38cedlinkissue5748 superseder
2018-03-07 00:11:14roundup-botsetmessages: + msg38839
2018-03-07 00:10:17roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg38838
2018-02-13 13:15:25reviewbotsetmessages: + msg38337
2018-02-13 13:15:18reviewbotsetnosy: + reviewbot
messages: + msg38336
2018-02-13 12:53:08cedsetstatus: in-progress -> testing
reviews: 41051002,41061002
keyword: + review
2018-02-13 12:40:55cedcreate

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