Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033966Openbravo ERPI. Performancepublic2013-06-26 11:062016-09-16 09:17
Reporterjecharri 
Assigned Todmiguelez 
PriorityurgentSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR16Q3.2Fixed in Version3.0PR16Q3.2 
Merge Request Status
Review Assigned Toaferraz
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033966: Accounting background process has bad performance
DescriptionAccounting background process has bad performance

With huge quantity of documents to be posted the accounting process can take much time.

More than 24 hours
Steps To Reproduce-
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks design defect 0024191 closed dmiguelez Accounting background process has bad performance 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-09-13 12:52dmiguelezTypedesign defect => backport
2016-09-13 12:52dmiguelezTarget Version => 3.0PR16Q3.2
2016-09-16 09:15hgbotCheckin
2016-09-16 09:15hgbotNote Added: 0090010
2016-09-16 09:15hgbotStatusscheduled => resolved
2016-09-16 09:15hgbotResolutionopen => fixed
2016-09-16 09:15hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR16Q3.2/rev/fea703d23f845a9bbccae7c2355e4a5a1dc1e4c9 [^]
2016-09-16 09:15hgbotCheckin
2016-09-16 09:15hgbotNote Added: 0090011
2016-09-16 09:15hgbotCheckin
2016-09-16 09:15hgbotNote Added: 0090012
2016-09-16 09:17aferrazReview Assigned To => aferraz
2016-09-16 09:17aferrazNote Added: 0090017
2016-09-16 09:17aferrazStatusresolved => closed
2016-09-16 09:17aferrazFixed in Version => 3.0PR16Q3.2

Notes
(0090010)
hgbot   
2016-09-16 09:15   
Repository: erp/backports/3.0PR16Q3.2
Changeset: fea703d23f845a9bbccae7c2355e4a5a1dc1e4c9
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Thu Sep 08 16:36:16 2016 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR16Q3.2/rev/fea703d23f845a9bbccae7c2355e4a5a1dc1e4c9 [^]

Fixes Issue 33966. Improves performance for Accounting Background Process.

Adds Index in FIN_FINACC_TRANSACTION Table.
Modifies query in AcctServer_data to properly limit the results returned.
Modifies query in AcctServer_data to remove the order clause.
Persist information about Accounting Schemas in Java code so it is no longer
necessary to access the Database everytime to retrive that information.

---
M src-db/database/model/tables/FIN_FINACC_TRANSACTION.xml
M src/org/openbravo/dal/core/OBContext.java
M src/org/openbravo/erpCommon/ad_forms/AcctSchema.java
M src/org/openbravo/erpCommon/ad_forms/AcctServer.java
M src/org/openbravo/erpCommon/ad_forms/AcctServer_data.xsql
A src/org/openbravo/erpCommon/ad_forms/AcctSchemaStructureProvider.java
---
(0090011)
hgbot   
2016-09-16 09:15   
Repository: erp/backports/3.0PR16Q3.2
Changeset: c5495a086bf8177dc5e235fa87a8ebe72888df0e
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Tue Sep 13 18:24:02 2016 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR16Q3.2/rev/c5495a086bf8177dc5e235fa87a8ebe72888df0e [^]

Related to issue 33966: Code review improvements

- Move AcctSchemaStructureProvider.java to same folder as OrganizationStructureProvider.java.
- Rename acctSchemaSP variable to acctSchemaProvider in OBContext.java.
- Remove unused data variable in AcctSchema.java.
- Add commentary to explain why we send limit manually to xsql query in AcctServer.java.
- Do not send numberRegisters parameter to AcctServerData.select as we are already sending the limit.
- Undo changes in AcctServerData.periodOpen method and create new AcctServerData.selectPeriodOpen method to avoid a functional API change.

---
M src/org/openbravo/dal/core/OBContext.java
M src/org/openbravo/erpCommon/ad_forms/AcctSchema.java
M src/org/openbravo/erpCommon/ad_forms/AcctServer.java
M src/org/openbravo/erpCommon/ad_forms/AcctServer_data.xsql
A src/org/openbravo/dal/security/AcctSchemaStructureProvider.java
R src/org/openbravo/erpCommon/ad_forms/AcctSchemaStructureProvider.java
---
(0090012)
hgbot   
2016-09-16 09:15   
Repository: erp/backports/3.0PR16Q3.2
Changeset: aea815e75a2f93c56fce5164e0f522a79d997ef4
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Wed Sep 14 18:24:33 2016 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR16Q3.2/rev/aea815e75a2f93c56fce5164e0f522a79d997ef4 [^]

Related to issue 33966: Update copyright

---
M src/org/openbravo/erpCommon/ad_forms/AcctServer_data.xsql
---
(0090017)
aferraz   
2016-09-16 09:17   
Code review + Testing OK