Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0040242Openbravo ERP09. Financial managementpublic2019-02-14 16:062019-02-20 15:11
Reporterjoniturralde93 
Assigned ToAtulOpenbravo 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0PR18Q3.3 
Target Version3.0PR18Q3.5Fixed in Version3.0PR18Q3.5 
Merge Request Status
Review Assigned ToSandrahuguet
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0040242: Unpost process can be very slow
DescriptionIn a customer instance with a lot of data in FACT_ACCT, the queries are too slow when filtering by record_id.
Attached query from ResetAccounting (getDocbasetypes method) which makes the process slow.

Also attached explain analyze before and after our proposed solution.
Steps To ReproduceIn customer environment:

1. Go to goods shipment.
2. Unpost a record.
3. Process time can vary, but sometimes can easily take 30-60 mins

Almost all the time consumed by the attached query.

Proposed SolutionCREATE INDEX FACT_ACCT_RECORDID ON FACT_ACCT(recordid);
Additional Information
TagsNo tags attached.
Relationships
blocks defect 00402083.0PR18Q3.3 closed AtulOpenbravo Unpost process can be very slow 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2019-02-20 10:11AtulOpenbravoTypedefect => backport
2019-02-20 10:11AtulOpenbravoTarget Version3.0PR18Q3.3 => 3.0PR18Q3.5
2019-02-20 15:10hgbotCheckin
2019-02-20 15:10hgbotNote Added: 0109957
2019-02-20 15:10hgbotStatusscheduled => resolved
2019-02-20 15:10hgbotResolutionopen => fixed
2019-02-20 15:10hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR18Q3.5/rev/2c2f81e031ba143d00e29465b706d8be62585379 [^]
2019-02-20 15:11SandrahuguetReview Assigned To => Sandrahuguet
2019-02-20 15:11SandrahuguetNote Added: 0109958
2019-02-20 15:11SandrahuguetStatusresolved => closed
2019-02-20 15:11SandrahuguetFixed in Version => 3.0PR18Q3.5

Notes
(0109957)
hgbot   
2019-02-20 15:10   
Repository: erp/backports/3.0PR18Q3.5
Changeset: 2c2f81e031ba143d00e29465b706d8be62585379
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Wed Feb 20 09:46:02 2019 +0530
URL: http://code.openbravo.com/erp/backports/3.0PR18Q3.5/rev/2c2f81e031ba143d00e29465b706d8be62585379 [^]

Fixes Issue 40242:Unpost process can be very slow

To use existing index fact_acct_table_record_id for
ad_table_id and record_id columns of fact_acct, ad_table_id
is added in AND clause of exists subquery from fact_acct
table only if record id is sent to getDocbasetypes method.

---
M src/org/openbravo/financial/ResetAccounting.java
---
(0109958)
Sandrahuguet   
2019-02-20 15:11   
Code review + testing OK