Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0034599Openbravo ERP09. Financial managementpublic2016-11-22 18:112016-12-16 18:39
maite 
AtulOpenbravo 
urgentmajoralways
closedfixed 
5
 
3.0PR17Q1 
aferraz
Core
No
0034599: Performance problem in AcctServer process when posting a reconciliation with aprox 800 lines
Process takes aprox 1.5h to post reconciliation document
I can provide environment where problem is reproducible
No tags attached.
Issue History
2016-11-22 18:11maiteNew Issue
2016-11-22 18:11maiteAssigned To => Triage Finance
2016-11-22 18:11maiteModules => Core
2016-11-22 18:11maiteResolution time => 1480374000
2016-11-22 18:11maiteTriggers an Emergency Pack => No
2016-11-22 18:12maiteIssue Monitored: networkb
2016-11-22 19:16aferrazAssigned ToTriage Finance => AtulOpenbravo
2016-11-23 08:13heccamIssue Monitored: heccam
2016-11-24 17:36aferrazResolution time1480374000 => 1481583600
2016-11-24 17:59AtulOpenbravoStatusnew => scheduled
2016-12-15 12:59hgbotCheckin
2016-12-15 12:59hgbotNote Added: 0092413
2016-12-15 12:59hgbotStatusscheduled => resolved
2016-12-15 12:59hgbotResolutionopen => fixed
2016-12-15 12:59hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/80dac576712ce6a9cc24a18c483fe6d726744075 [^]
2016-12-15 12:59hgbotCheckin
2016-12-15 12:59hgbotNote Added: 0092414
2016-12-15 13:00aferrazReview Assigned To => aferraz
2016-12-15 13:00aferrazNote Added: 0092415
2016-12-15 13:00aferrazStatusresolved => closed
2016-12-15 13:00aferrazFixed in Version => 3.0PR17Q1
2016-12-15 13:03aferrazNote Added: 0092416
2016-12-16 18:39hudsonbotCheckin
2016-12-16 18:39hudsonbotNote Added: 0092720
2016-12-16 18:39hudsonbotCheckin
2016-12-16 18:39hudsonbotNote Added: 0092721

Notes
(0092413)
hgbot   
2016-12-15 12:59   
Repository: erp/devel/pi
Changeset: 80dac576712ce6a9cc24a18c483fe6d726744075
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Wed Dec 14 15:54:12 2016 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/80dac576712ce6a9cc24a18c483fe6d726744075 [^]

Fixes issue 34599: Bad performance when posting reconciliation with many lines

** Add m_RecordID2 member variable, getter and setter method in FactLine.java
** Set RecordID2 while Creating FactLine in Fact.java
** After successfully creation of FactLine, add RecordID2 to Set.
** Move UpdateBalancedDate in Fact.java
** For every 1000 records in RecordID2 Set, call UpdateBalancedDate method
** Set of 1000 records is nescessary to meet Oracle In clause restrictions.

---
M src/org/openbravo/erpCommon/ad_forms/Fact.java
M src/org/openbravo/erpCommon/ad_forms/FactLine.java
M src/org/openbravo/erpCommon/ad_forms/FactLine_data.xsql
---
(0092414)
hgbot   
2016-12-15 12:59   
Repository: erp/devel/pi
Changeset: ebd14e85e73d2dce2268d8254afbfc9c83bff61a
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Thu Dec 15 12:39:26 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/ebd14e85e73d2dce2268d8254afbfc9c83bff61a [^]

Related to issue 34599: Code review improvements

Improve splitRecordID2Set method.

---
M src/org/openbravo/erpCommon/ad_forms/Fact.java
---
(0092415)
aferraz   
2016-12-15 13:00   
Code review + Testing OK
(0092416)
aferraz   
2016-12-15 13:03   
Process has been improved from 1.5 hours to 18 seconds.
(0092720)
hudsonbot   
2016-12-16 18:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/dc8bf00badd0 [^]
Maturity status: Test
(0092721)
hudsonbot   
2016-12-16 18:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/dc8bf00badd0 [^]
Maturity status: Test