Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033182Openbravo ERP04. Warehouse managementpublic2016-06-07 18:412016-06-07 18:41
Reportershuehner 
Assigned ToTriage Omni WMS 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusnewResolutionopen 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033182: StockUtils.getStock functions forces flush on all users even if that is not wanted
DescriptionThe implementation of that function forces a flush to happen (even requesting 2 flushes). (indirectly via the 2 'true' in getConnection and New DalConnectionProvider)

For any user of the function must take that performance hit even it not required in a specific flow.
I.e. POS Orderloader needs a single flush before is 'various' getStock calls but currently one happens per call (inside getStock).

As that function is part of API a new method needs to be added for that.

I.e. a getStockNoFlush to make explicit to the caller that no flush is automatically done and inside flip both boolean's to false.
Steps To Reproduce-
Proposed Solution
Additional Information
TagsPerformance
Relationships
related to defect 0033181 new Retail Retail Modules Most OrderLoader insert flush() 'ed' during call to getStock which is non-obvious and breaks debug timing 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-06-07 18:41shuehnerNew Issue
2016-06-07 18:41shuehnerAssigned To => Triage Finance
2016-06-07 18:41shuehnerModules => Core
2016-06-07 18:41shuehnerTriggers an Emergency Pack => No
2016-06-07 18:41shuehnerTag Attached: Performance
2016-06-07 18:41shuehnerRelationship addedrelated to 0033181

There are no notes attached to this issue.