Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0020511Openbravo ERPA. Platformpublic2012-05-15 15:182012-05-25 12:12
Reporteralostale 
Assigned Toalostale 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Versionpi 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0020511: Unnecessary flush on CallProcess
DescriptionCallProcess is used to invoke PL processes using DAL. It performs an explicit flush to ensure current DAL transaction is in DB to be processed by PL. But the connection obtained for calling the process implicitly does an extra flush. This affects performance without any benefit.
Steps To ReproduceComplete an invoice from UI and check 2 flushes are performed.
Proposed Solution
Additional Information
TagsPerformance
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-05-15 15:18alostaleNew Issue
2012-05-15 15:18alostaleAssigned To => alostale
2012-05-15 15:18alostaleModules => Core
2012-05-15 15:18alostaleOBNetwork customer => No
2012-05-15 15:18alostaleTag Attached: Performance
2012-05-15 15:20alostaleNote Added: 0048649
2012-05-15 15:22hgbotCheckin
2012-05-15 15:22hgbotNote Added: 0048650
2012-05-15 15:22hgbotStatusnew => resolved
2012-05-15 15:22hgbotResolutionopen => fixed
2012-05-15 15:22hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/10639f3fc93441a19189faffba04504ffcb84ab6 [^]
2012-05-21 12:10migueldejuanaNote Added: 0048797
2012-05-21 12:10migueldejuanaStatusresolved => closed
2012-05-21 12:10migueldejuanaFixed in Version => pi
2012-05-25 12:12hudsonbotCheckin
2012-05-25 12:12hudsonbotNote Added: 0049131

Notes
(0048649)
alostale   
2012-05-15 15:20   
This fix is not risky as removes a flush done immediately after another one.
(0048650)
hgbot   
2012-05-15 15:22   
Repository: erp/devel/pi
Changeset: 10639f3fc93441a19189faffba04504ffcb84ab6
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue May 15 15:22:24 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/10639f3fc93441a19189faffba04504ffcb84ab6 [^]

fixed bug 20511: Unnecessary flush on CallProcess

---
M src/org/openbravo/service/db/CallProcess.java
---
(0048797)
migueldejuana   
2012-05-21 12:10   
Code reviewed and verified on pi@87340d8f38a3
(0049131)
hudsonbot   
2012-05-25 12:12   
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/5401e185a8b0 [^]

Maturity status: Test