Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
| ID | ||||||||||||
| 0043416 | ||||||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||||||
| defect | [Openbravo ERP] 04. Warehouse management | minor | have not tried | 2020-03-04 14:34 | 2020-03-04 14:35 | |||||||
| Reporter | shuehner | View Status | public | |||||||||
| Assigned To | Triage Omni WMS | |||||||||||
| Priority | normal | Resolution | open | Fixed in Version | ||||||||
| Status | new | Fix in branch | Fixed in SCM revision | |||||||||
| Projection | none | ETA | none | Target Version | ||||||||
| OS | Any | Database | Any | Java version | ||||||||
| OS Version | Database version | Ant version | ||||||||||
| Product Version | SCM revision | |||||||||||
| Review Assigned To | ||||||||||||
| Web browser | ||||||||||||
| Modules | Core | |||||||||||
| Regression level | ||||||||||||
| Regression date | ||||||||||||
| Regression introduced in release | ||||||||||||
| Regression introduced by commit | ||||||||||||
| Triggers an Emergency Pack | No | |||||||||||
| Summary | 0043416: PL-function m_get_product_cost could be made 2 * faster | |||||||||||
| Description | This pl-function checks twice in m_costing table: a.) First to see if there are any rows in the matching range b.) Then querying again to fetch the first interesting rows. In practice that doubles runtime compared to only doing a single query. Attached files (of OLD 2.50 version of the pl) show idea of this change. Pending to check: 1.) The date filters are not identical between the 2 checks. The 'BETWEEN' usage filters for <= DATETO while the fetching query does < DATETO That should be still fine as the 'shortcut count(*) is less restrictive' so the 2nd query should still return the correct result 2.) Recent change added extra filter condition: 17092: AND costtype NOT IN ('AVA' , 'STA') to make it 'not costing new engine costs'. That should be probably added instead to the 2nd query 3.) That change 17092 is unclear if totally correct today. While it does exclude AVA,STA from the shortcut query it will not filter those out when retrieving data. | |||||||||||
| Steps To Reproduce | - | |||||||||||
| Tags | No tags attached. | |||||||||||
| Attached Files |  m_get_product_cost-original.sql [^] (2,725 bytes) 2020-03-04 14:34 m_get_product_cost-new.sql [^] (2,457 bytes) 2020-03-04 14:34 | |||||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0118322) shuehner (administrator) 2020-03-04 14:35 |
Note: Issue is mostly reported to see if that improvement is still relevant / interesting with current 3.0 code. |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2020-03-04 14:34 | shuehner | New Issue | |
| 2020-03-04 14:34 | shuehner | Assigned To | => Triage Finance |
| 2020-03-04 14:34 | shuehner | Modules | => Core |
| 2020-03-04 14:34 | shuehner | Triggers an Emergency Pack | => No |
| 2020-03-04 14:34 | shuehner | File Added: m_get_product_cost-original.sql | |
| 2020-03-04 14:34 | shuehner | File Added: m_get_product_cost-new.sql | |
| 2020-03-04 14:35 | shuehner | Note Added: 0118322 | |
| 2020-03-04 18:17 | shuehner | Relationship added | related to 0022417 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |