Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0030516Openbravo ERP04. Warehouse managementpublic2015-08-07 12:102015-08-20 23:16
Reporteralostale 
Assigned ToAtulOpenbravo 
PriorityimmediateSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tovmromanos
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2015-02-15
Regression introduced in release3.0PR15Q2
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/b4be2e4511445d578fc7f8a2175c3a6114e52583 [^]
Triggers an Emergency PackNo
Summary0030516: Potential full c_orderline scan in M_Reservation_trg
DescriptionTrigger on M_Reservation table does a count on all order lines without any filtering.

If this code is executed in an environment with a decent amount of orders, it can take ages to complete.





Steps To ReproduceI have not reproduced it, but just read the code in M_RESERVATION_TRG.xml :

https://code.openbravo.com/erp/devel/pi/rev/b4be2e4511445d578fc7f8a2175c3a6114e52583#l1.28 [^]

select count (*) into v_count from c_orderline;

it looks like some missing debug code forgotten to remove as it is never used
Proposed Solution
Additional Information
TagsPerformance
Relationships
depends on backport 00305283.0PR15Q3 closed AtulOpenbravo Potential full c_orderline scan in M_Reservation_trg 
depends on backport 00305293.0PR15Q2.4 closed AtulOpenbravo Potential full c_orderline scan in M_Reservation_trg 
caused by defect 0029063 closed umartirena M_Reservation_Post fails in Oracle 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-08-07 12:10alostaleNew Issue
2015-08-07 12:10alostaleAssigned To => Triage Finance
2015-08-07 12:10alostaleOBNetwork customer => No
2015-08-07 12:10alostaleModules => Core
2015-08-07 12:10alostaleRegression level => Production - Confirmed Stable
2015-08-07 12:10alostaleRegression date => 2015-02-15
2015-08-07 12:10alostaleRegression introduced in release => 3.0PR15Q2
2015-08-07 12:10alostaleRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/b4be2e4511445d578fc7f8a2175c3a6114e52583 [^]
2015-08-07 12:10alostaleTriggers an Emergency Pack => No
2015-08-07 12:10alostaleTag Attached: Performance
2015-08-07 12:10alostaleRelationship addedcaused by 0029063
2015-08-07 12:12alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=9178#r9178
2015-08-10 06:50AtulOpenbravoAssigned ToTriage Finance => AtulOpenbravo
2015-08-10 06:51AtulOpenbravoStatusnew => scheduled
2015-08-10 09:13hgbotCheckin
2015-08-10 09:13hgbotNote Added: 0079271
2015-08-10 09:13hgbotStatusscheduled => resolved
2015-08-10 09:13hgbotResolutionopen => fixed
2015-08-10 09:13hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/17b020a23cb9d5a9aacf9828773a303356276751 [^]
2015-08-10 09:14vmromanosReview Assigned To => vmromanos
2015-08-10 09:14vmromanosStatusresolved => closed
2015-08-20 23:16hudsonbotCheckin
2015-08-20 23:16hudsonbotNote Added: 0079536

Notes
(0079271)
hgbot   
2015-08-10 09:13   
Repository: erp/devel/pi
Changeset: 17b020a23cb9d5a9aacf9828773a303356276751
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Sun Aug 09 09:01:40 2015 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/17b020a23cb9d5a9aacf9828773a303356276751 [^]

Fixes Issue 30516:Potential full c_orderline scan in M_Reservation_trg

Removed unwanted line in code which fully scans c_orderline.

---
M src-db/database/model/triggers/M_RESERVATION_TRG.xml
---
(0079536)
hudsonbot   
2015-08-20 23:16   
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/8c91718397a5 [^]
Maturity status: Test