Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0025682 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] I. Performance | major | have not tried | 2014-02-07 07:46 | 2014-02-07 14:09 | |||
Reporter | shuehner | View Status | public | |||||
Assigned To | shuehner | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | c73830693d65 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | 3.0MP22.2 | SCM revision | ||||||
Merge Request Status | ||||||||
Review Assigned To | dmiguelez | |||||||
OBNetwork customer | OBPS | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0025682: Missing index for c_invoice table c_order_id column makes at least c_order_trg + c_order_trg2 slow | |||||||
Description | Both those triggers do an update to c_invoice which is looked up via c_invoice.c_order_id to synchronize a few fields from c_order to c_invoice. with 700k rows in c_invoice just this one update is taking 230ms each time. | |||||||
Steps To Reproduce | Observe trigger runtime in database with many invoices. i.e. set track_functions = 'all' in pg config Then after running code executing the update check trigger runtime with select * from pg_stat_user_functions order by total_time desc | |||||||
Proposed Solution | Add proposed trigger create index c_invoice_order on c_invoice(c_order_id); | |||||||
Tags | Performance | |||||||
Attached Files | ||||||||
![]() |
||||||||
|
![]() |
|
(0064009) shuehner (administrator) 2014-02-07 07:52 |
I think this fix should be included in the next MP30.x emergency mp as i see no regression risk and it is a useful performance fix to get out to every user. |
(0064016) hgbot (developer) 2014-02-07 14:01 |
Repository: erp/devel/pi Changeset: c73830693d6559bcdcc997321ad3d93f47436f6b Author: Stefan Hühner <stefan.huehner <at> openbravo.com> Date: Fri Feb 07 12:56:56 2014 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/c73830693d6559bcdcc997321ad3d93f47436f6b [^] Fixed 25682: Add index on c_invoice.c_order_id to speedup triggers c_order_trg & c_order_trg2 --- M src-db/database/model/tables/C_INVOICE.xml --- |
(0064017) dmiguelez (viewer) 2014-02-07 14:09 |
Code Review + Testing Ok |
![]() |
|||
Date Modified | Username | Field | Change |
2014-02-07 07:46 | shuehner | New Issue | |
2014-02-07 07:46 | shuehner | Assigned To | => AugustoMauch |
2014-02-07 07:46 | shuehner | Modules | => Core |
2014-02-07 07:46 | shuehner | OBNetwork customer | => Yes |
2014-02-07 07:46 | shuehner | Triggers an Emergency Pack | => No |
2014-02-07 07:49 | shuehner | Relationship added | related to 0025683 |
2014-02-07 07:52 | shuehner | Note Added: 0064009 | |
2014-02-07 08:03 | alostale | Tag Attached: Performance | |
2014-02-07 08:03 | alostale | Assigned To | AugustoMauch => dmiguelez |
2014-02-07 12:33 | priyam | Issue Monitored: priyam | |
2014-02-07 14:01 | hgbot | Checkin | |
2014-02-07 14:01 | hgbot | Note Added: 0064016 | |
2014-02-07 14:01 | hgbot | Status | new => resolved |
2014-02-07 14:01 | hgbot | Resolution | open => fixed |
2014-02-07 14:01 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/c73830693d6559bcdcc997321ad3d93f47436f6b [^] |
2014-02-07 14:01 | shuehner | Assigned To | dmiguelez => shuehner |
2014-02-07 14:09 | dmiguelez | Review Assigned To | => dmiguelez |
2014-02-07 14:09 | dmiguelez | Note Added: 0064017 | |
2014-02-07 14:09 | dmiguelez | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |