Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0040240Openbravo ERP09. Financial managementpublic2019-02-14 16:062019-02-20 16:38
Reporterjoniturralde93 
Assigned ToAtulOpenbravo 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0PR18Q3.3 
Target Version3.0PR19Q1Fixed in Version3.0PR19Q1 
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
Summary0040240: 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.0PR19Q1
2019-02-20 16:38hgbotCheckin
2019-02-20 16:38hgbotNote Added: 0109962
2019-02-20 16:38hgbotStatusscheduled => resolved
2019-02-20 16:38hgbotResolutionopen => fixed
2019-02-20 16:38hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR19Q1/rev/42d7938062af4385456d59792ed69b6460cd5ed3 [^]
2019-02-20 16:38SandrahuguetReview Assigned To => Sandrahuguet
2019-02-20 16:38SandrahuguetNote Added: 0109963
2019-02-20 16:38SandrahuguetStatusresolved => closed
2019-02-20 16:38SandrahuguetFixed in Version => 3.0PR19Q1

Notes
(0109962)
hgbot   
2019-02-20 16:38   
Repository: erp/backports/3.0PR19Q1
Changeset: 42d7938062af4385456d59792ed69b6460cd5ed3
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.0PR19Q1/rev/42d7938062af4385456d59792ed69b6460cd5ed3 [^]

Fixes Issue 0040240: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
---
(0109963)
Sandrahuguet   
2019-02-20 16:38   
Code review + testing OK