Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034599Openbravo ERP09. Financial managementpublic2016-11-22 18:112016-12-16 18:39
Reportermaite 
Assigned ToAtulOpenbravo 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR17Q1 
Merge Request Status
Review Assigned Toaferraz
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket44532
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034599: Performance problem in AcctServer process when posting a reconciliation with aprox 800 lines
DescriptionProcess takes aprox 1.5h to post reconciliation document
Steps To ReproduceI can provide environment where problem is reproducible
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-11-22 18:11maiteNew Issue
2016-11-22 18:11maiteAssigned To => Triage Finance
2016-11-22 18:11maiteOBNetwork customer => Yes
2016-11-22 18:11maiteModules => Core
2016-11-22 18:11maiteSupport ticket => 44532
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