Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0037236 | Openbravo ERP | 03. Procurement management | public | 2017-11-06 17:27 | 2018-01-03 12:38 |
|
| Reporter | gorkaion | |
| Assigned To | dmiguelez | |
| Priority | urgent | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | 3.0PR18Q1 | |
| Merge Request Status | |
| Review Assigned To | vmromanos |
| OBNetwork customer | |
| Web browser | |
| Modules | Core |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0037236: Create Lines From popup on Purchase Invoice slow when having many Orders |
| Description | On a environment with a high volume of Orders. Try to open the Create Lines From in Purchase Invoice window.
The popup can take up to 30 seconds to Open. Selecting a Shipment or Receipt the page is reloaded taking another 30 seconds. |
| Steps To Reproduce | Get an environment with a high volume of orders. ~1 million
Have a vendor with some open orders pending to invoice.
Try to open the Create Lines From window. |
| Proposed Solution | The issue is in the Query to load the combo of Purchase Orders.
selectFromPOCombo of CreateFromInvoice_data.xsql
This query is doing a Seq Scan on M_MatchPO. This can be avoided by an index on c_orderline_id similar to the existing one on m_inoutline_id
"m_matchpo_inout" btree (m_inoutline_id) WHERE m_inoutline_id IS NOT NULL
Consider if the new index can replace the "m_matchpo_ship" index. |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files | |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2017-11-06 17:27 | gorkaion | New Issue | |
| 2017-11-06 17:27 | gorkaion | Assigned To | => Triage Finance |
| 2017-11-06 17:27 | gorkaion | Modules | => Core |
| 2017-11-06 17:27 | gorkaion | Resolution time | => 1511737200 |
| 2017-11-06 17:27 | gorkaion | Triggers an Emergency Pack | => No |
| 2017-11-06 17:50 | dmiguelez | Assigned To | Triage Finance => dmiguelez |
| 2017-11-07 11:23 | dmiguelez | Note Added: 0100301 | |
| 2017-11-07 17:15 | hgbot | Checkin | |
| 2017-11-07 17:15 | hgbot | Note Added: 0100311 | |
| 2017-11-07 17:15 | hgbot | Status | new => resolved |
| 2017-11-07 17:15 | hgbot | Resolution | open => fixed |
| 2017-11-07 17:15 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/38268cab93c5aa3f60ea3bcfa812168eec308d44 [^] |
| 2017-11-07 18:01 | vmromanos | Review Assigned To | => vmromanos |
| 2017-11-07 18:01 | vmromanos | Note Added: 0100312 | |
| 2017-11-07 18:01 | vmromanos | Status | resolved => closed |
| 2017-11-08 10:39 | dmiguelez | Fixed in Version | => 3.0PR18Q1 |
| 2018-01-03 12:38 | hudsonbot | Checkin | |
| 2018-01-03 12:38 | hudsonbot | Note Added: 0101378 | |
|
Notes |
|
|
|
Tested with client's data.
Without the patch took 30 seconds.
With the patch less than a second. |
|
|
|
(0100311)
|
|
hgbot
|
|
2017-11-07 17:15
|
|
Repository: erp/devel/pi
Changeset: 38268cab93c5aa3f60ea3bcfa812168eec308d44
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Tue Nov 07 11:20:04 2017 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/38268cab93c5aa3f60ea3bcfa812168eec308d44 [^]
Fixes Issue 37236. Modifies index over c_orderline and m_inoutline
to remove the clause "where m_inoutline_id is null"
By doing so, the c_orderline is fully indexed and improves performance.
There is no need to create a partial index for the c_orderline,
since this column is mandatory.
---
M src-db/database/model/tables/M_MATCHPO.xml
---
|
|
|
|
|
|
|
|
|
|