Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0027160Openbravo ERP07. Sales managementpublic2014-07-23 08:252015-02-24 16:53
Reporteralostale 
Assigned Toeduardo_Argal 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR15Q2 
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
Summary0027160: C_Order_Post poor performance
DescriptionThis is an umbrella issue to keep track of all known problems regarding C_Order_Post performance.
Steps To Reproduce-Enable stock reservations
-Create a Sales Order with 1200 lines all of them creating reservations
-Book it -> takes 5min

See more metrics here: https://docs.google.com/a/openbravo.com/spreadsheets/d/10vAhiNngnOvPChLvdLq-4nDSWnL6IMaQ5MU2IWEi5fM/edit?usp=sharing [^]
Proposed Solution
Additional Information
TagsPerformance
Relationships
depends on defect 0027022 closed alostale Openbravo ERP Sequential scans in ad_preference table 
depends on defect 0027023 closed vmromanos Openbravo ERP Order line taxes are regenerated on all order line changes 
depends on defect 0027024 closed vmromanos Openbravo ERP Unneeded updates to c_order whenever c_orderline is modified 
depends on defect 00270263.0PR15Q2 closed eduardo_Argal Openbravo ERP Poor performance of m_get_stock 
depends on defect 00271403.0PR14Q4 closed vmromanos Openbravo ERP Useless usage of hex_to_int() in C_ORDER_POST affects performance 
depends on defect 00271413.0PR14Q4 closed vmromanos Openbravo ERP Minor performance fix in C_ORDERLINETAX_TRG 
depends on defect 0027101 closed eduardo_Argal Openbravo ERP sequential scans on reservation tables on m_get_stock function execution 
related to defect 00290733.0PR15Q2 closed umartirena Openbravo ERP Review deleting logic of tax lines in C_INVOICELINE_TRG2 
related to defect 0029177 closed eduardo_Argal Modules The sequence for picking list on ad_sequence table is locked while the picking list process is executed 
related to defect 0028833RR15Q2 closed mario_castello Retail Modules The OrderLoader needs to use the new way of calling to the M_GetStock PL function 
related to defect 0030756 new Triage Omni WMS Openbravo ERP Documentation about how to create a warehouse rule is missing 
related to feature request 0031409 new Triage Omni WMS Modules It would be fine if a picking list could be created for an on hand warehouse different from the order's warehouse 
related to defect 0036091 closed AtulOpenbravo Openbravo ERP ProcessID parameter is not available in the extension points of M_GET_STOCK_PARAM function 
blocks defect 00272103.0PR15Q2 closed AtulOpenbravo Openbravo ERP Records in m_stock_proposed table should be removed once they are used 
blocks defect 0027027 closed eduardo_Argal Modules Generate Picking List process doesn't scale 
causes defect 0029218 closed eduardo_Argal Openbravo ERP Cannot save a Stock Reservation in PostgreSQL 8.4 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-07-23 08:25alostaleNew Issue
2014-07-23 08:25alostaleAssigned To => dmiguelez
2014-07-23 08:25alostaleModules => Core
2014-07-23 08:25alostaleTriggers an Emergency Pack => No
2014-07-23 08:25alostaleTag Attached: Performance
2014-07-23 08:25alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=6120#r6120
2014-07-23 08:26alostaleRelationship addeddepends on 0027022
2014-07-23 08:26alostaleRelationship addeddepends on 0027023
2014-07-23 08:26alostaleRelationship addeddepends on 0027024
2014-07-23 08:26alostaleRelationship addeddepends on 0027026
2014-07-23 08:26alostaleRelationship addeddepends on 0027140
2014-07-23 08:26alostaleRelationship addeddepends on 0027141
2014-07-23 08:28alostaleRelationship addeddepends on 0027101
2015-02-02 12:29hgbotCheckin
2015-02-02 12:29hgbotNote Added: 0073980
2015-02-02 12:29hgbotStatusnew => resolved
2015-02-02 12:29hgbotResolutionopen => fixed
2015-02-02 12:29hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/7091c40b9be09114b26b6cefeb45bc30c2650bf3 [^]
2015-02-02 12:29hgbotCheckin
2015-02-02 12:29hgbotNote Added: 0073981
2015-02-02 13:15hgbotCheckin
2015-02-02 13:15hgbotNote Added: 0073985
2015-02-02 15:53SandrahuguetReview Assigned To => Sandrahuguet
2015-02-02 15:53SandrahuguetNote Added: 0073989
2015-02-02 15:53SandrahuguetStatusresolved => closed
2015-02-02 15:53SandrahuguetFixed in Version => 3.0PR15Q2
2015-02-02 16:14hgbotCheckin
2015-02-02 16:14hgbotNote Added: 0073991
2015-02-03 13:11jonalegriaesarteRelationship addedblocks 0027210
2015-02-04 20:35hudsonbotCheckin
2015-02-04 20:35hudsonbotNote Added: 0074203
2015-02-04 20:35hudsonbotCheckin
2015-02-04 20:35hudsonbotNote Added: 0074204
2015-02-04 20:35hudsonbotCheckin
2015-02-04 20:35hudsonbotNote Added: 0074205
2015-02-24 16:52SandrahuguetRelationship addedblocks 0027027
2015-02-24 16:53SandrahuguetAssigned Todmiguelez => eduardo_Argal
2015-02-27 11:25dmitry_mezentsevRelationship addedrelated to 0029073
2015-03-06 15:36egoitzRelationship addedrelated to 0029177
2015-03-10 12:57ngarciaRelationship addedcauses 0029218
2015-06-14 11:27dmitry_mezentsevRelationship addedrelated to 0028833
2015-10-09 10:32ngarciaRelationship addedrelated to 0030756
2015-11-09 17:54ngarciaRelationship addedrelated to 0031409
2017-05-24 16:48ngarciaRelationship addedrelated to 0036091

Notes
(0073980)
hgbot   
2015-02-02 12:29   
Repository: erp/devel/pi
Changeset: 7091c40b9be09114b26b6cefeb45bc30c2650bf3
Author: Eduardo Argal Guibert <eduardo.argal <at> openbravo.com>
Date: Wed Jan 21 13:14:46 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/7091c40b9be09114b26b6cefeb45bc30c2650bf3 [^]

Fixed issue 27160: C_Order_Post poor performance

---
M referencedata/sampledata/F_B_International_Group/M_WAREHOUSE_RULE.xml
M referencedata/sampledata/QA_Testing/M_WAREHOUSE_RULE.xml
M src-db/database/model/functions/MA_PRODUCTIONRUN_STANDARD.xml
M src-db/database/model/functions/MRP_RUN_INITIALIZE.xml
M src-db/database/model/functions/M_GET_STOCK.xml
M src-db/database/model/functions/M_INOUT_CREATE.xml
M src-db/database/model/functions/M_PRODUCTION_RUN.xml
M src-db/database/model/functions/M_RESERVATION_POST.xml
M src-db/database/model/functions/M_RESERVATION_REALLOCATE.xml
M src-db/database/model/functions/M_RESERVE_STOCK_AUTO.xml
M src-db/database/model/functions/M_RESERVE_STOCK_MANUAL.xml
M src-db/database/model/tables/AD_EXTENSION_POINTS.xml
M src-db/database/model/tables/M_STOCK_PROPOSED.xml
M src-db/database/model/tables/M_STORAGE_DETAIL.xml
M src-db/database/model/tables/M_WAREHOUSE_RULE.xml
M src-db/database/model/triggers/M_RESERVATION_STOCK_TRG.xml
M src-db/database/model/triggers/M_RESERVATION_TRG.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_EXTENSION_POINTS.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src-db/database/sourcedata/AD_REFERENCE.xml
M src-db/database/sourcedata/AD_REF_LIST.xml
M src/org/openbravo/dal/core/OBContext.java
M src/org/openbravo/erpCommon/utility/OBMessageUtils.java
M src/org/openbravo/materialmgmt/ReservationUtils.java
A src-db/database/model/functions/M_GET_STOCK_PARAM.xml
A src-util/modulescript/src/org/openbravo/modulescript/InitializeReservationColumnsForStorageDetail.java
A src-util/modulescript/src/org/openbravo/modulescript/InitializeReservationColumnsForStorageDetail_data.xsql
---
(0073981)
hgbot   
2015-02-02 12:29   
Repository: erp/devel/pi
Changeset: aa854421dcf59326755b80be2ac2c0bd081cb125
Author: Sandra Huguet <sandra.huguet <at> openbravo.com>
Date: Fri Jan 30 11:44:11 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/aa854421dcf59326755b80be2ac2c0bd081cb125 [^]

Related to issue 27160 code review

fixed pls to work with oracle, update Copyright,
add missing .class files.

---
M src-db/database/model/functions/MA_PRODUCTIONRUN_STANDARD.xml
M src-db/database/model/functions/MRP_RUN_INITIALIZE.xml
M src-db/database/model/functions/M_GET_STOCK.xml
M src-db/database/model/functions/M_GET_STOCK_PARAM.xml
M src-db/database/model/functions/M_INOUT_CREATE.xml
M src-db/database/model/functions/M_PRODUCTION_RUN.xml
M src-db/database/model/functions/M_RESERVATION_POST.xml
M src-db/database/model/functions/M_RESERVATION_REALLOCATE.xml
M src-db/database/model/functions/M_RESERVE_STOCK_AUTO.xml
M src-db/database/model/functions/M_RESERVE_STOCK_MANUAL.xml
M src-db/database/model/triggers/M_RESERVATION_STOCK_TRG.xml
M src-util/modulescript/src/org/openbravo/modulescript/InitializeReservationColumnsForStorageDetail.java
M src-util/modulescript/src/org/openbravo/modulescript/InitializeReservationColumnsForStorageDetail_data.xsql
M src/org/openbravo/dal/core/OBContext.java
M src/org/openbravo/erpCommon/utility/OBMessageUtils.java
M src/org/openbravo/materialmgmt/ReservationUtils.java
A src-util/modulescript/build/classes/org/openbravo/modulescript/InitializeReservationColumnsForStorageDetail.class
A src-util/modulescript/build/classes/org/openbravo/modulescript/InitializeReservationColumnsForStorageDetailData.class
---
(0073985)
hgbot   
2015-02-02 13:15   
Repository: erp/pmods/org.openbravo.warehouse.pickinglist
Changeset: e9af7705fa554a5291592d8d4c29000beb440efe
Author: Sandra Huguet <sandra.huguet <at> openbravo.com>
Date: Mon Feb 02 13:15:23 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.warehouse.pickinglist/rev/e9af7705fa554a5291592d8d4c29000beb440efe [^]

Related to issue 27160: C_Order_Post poor performance

necessary changes to pickinglist module

---
M src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
M src/org/openbravo/warehouse/pickinglist/Utilities.java
---
(0073989)
Sandrahuguet   
2015-02-02 15:53   
Code review + testing OK
(0073991)
hgbot   
2015-02-02 16:14   
Repository: erp/devel/pi
Changeset: cd23318092f743149890b7985f1591635553a4eb
Author: Sandra Huguet <sandra.huguet <at> openbravo.com>
Date: Mon Feb 02 16:12:43 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/cd23318092f743149890b7985f1591635553a4eb [^]

related to issue 27160 update Copyright

---
M src-db/database/model/functions/M_GET_STOCK_PARAM.xml
M src-util/modulescript/src/org/openbravo/modulescript/InitializeReservationColumnsForStorageDetail.java
M src-util/modulescript/src/org/openbravo/modulescript/InitializeReservationColumnsForStorageDetail_data.xsql
---
(0074203)
hudsonbot   
2015-02-04 20:35   
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/f36c91d0ad63 [^]
Maturity status: Test
(0074204)
hudsonbot   
2015-02-04 20:35   
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/f36c91d0ad63 [^]
Maturity status: Test
(0074205)
hudsonbot   
2015-02-04 20:35   
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/f36c91d0ad63 [^]
Maturity status: Test