Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0037236 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 03. Procurement management | major | always | 2017-11-06 17:27 | 2018-01-03 12:38 | |||
| Reporter | gorkaion | View Status | public | |||||
| Assigned To | dmiguelez | |||||||
| Priority | urgent | Resolution | fixed | Fixed in Version | 3.0PR18Q1 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | 38268cab93c5 | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | vmromanos | |||||||
| OBNetwork customer | OBPS | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | 49322 | |||||||
| 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. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0100301) dmiguelez (viewer) 2017-11-07 11:23 |
Tested with client's data. Without the patch took 30 seconds. With the patch less than a second. |
|
(0100311) hgbot (developer) 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 --- |
|
(0100312) vmromanos (viewer) 2017-11-07 18:01 |
Code review OK |
|
(0101378) hudsonbot (viewer) 2018-01-03 12:38 |
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/c81e0d3cbab5 [^] Maturity status: Test |
|
Notes |
|
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 | OBNetwork customer | => Yes |
| 2017-11-06 17:27 | gorkaion | Modules | => Core |
| 2017-11-06 17:27 | gorkaion | Support ticket | => 49322 |
| 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 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |