Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034513Openbravo ERP09. Financial managementpublic2016-11-15 09:182017-05-29 17:43
Reportercaristu 
Assigned ToAtulOpenbravo 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR17Q3 
Merge Request Status
Review Assigned Toaferraz
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034513: [clustering] CreateRegFactAcct is locked just at JVM level
DescriptionPrivate method processButton invoked from doPost in CreateRegFactAcct is a synchronized method. This means that the process has been designed to prevent its execution by different threads in parallel.

In a clustered environment the previous assumption is not guaranteed, as the synchronized method is preventing concurrent executions within the same JVM.

Therefore it could be possible to execute this process in parallel by different threads which belong to different nodes of a clustered environment.
Steps To ReproduceIn description
Proposed Solution
Additional Information
Tagsclustering
Relationships
has duplicate defect 0034515 closed collazoandy4 Unneeded lock in processButtonReg and processButtonClose methods of CreateRegFactAcct 
blocks design defect 0034490 acknowledged Triage Platform Base [clustering] Support clustering 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-11-15 09:18caristuNew Issue
2016-11-15 09:18caristuAssigned To => Triage Finance
2016-11-15 09:18caristuOBNetwork customer => No
2016-11-15 09:18caristuModules => Core
2016-11-15 09:18caristuTriggers an Emergency Pack => No
2016-11-15 09:20caristuRelationship addedblocks 0034490
2016-11-15 10:16caristuRelationship addeddepends on 0034516
2016-11-15 10:16caristuRelationship deleteddepends on 0034516
2016-11-15 10:17caristuDescription Updatedbug_revision_view_page.php?rev_id=13718#r13718
2016-11-15 11:13caristuTag Attached: clustering
2017-03-23 15:03markmm82Assigned ToTriage Finance => AtulOpenbravo
2017-05-24 08:52AtulOpenbravoStatusnew => scheduled
2017-05-24 10:11hgbotCheckin
2017-05-24 10:11hgbotNote Added: 0096745
2017-05-24 10:11hgbotStatusscheduled => resolved
2017-05-24 10:11hgbotResolutionopen => fixed
2017-05-24 10:11hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/e96d2416282f08afd7c412c182b762dc287d3726 [^]
2017-05-24 10:13aferrazReview Assigned To => aferraz
2017-05-24 10:13aferrazNote Added: 0096747
2017-05-24 10:13aferrazStatusresolved => closed
2017-05-24 10:13aferrazFixed in Version => 3.0PR17Q3
2017-05-29 17:43hudsonbotCheckin
2017-05-29 17:43hudsonbotNote Added: 0096982
2017-12-27 09:43vmromanosRelationship addedhas duplicate 0034515

Notes
(0096745)
hgbot   
2017-05-24 10:11   
Repository: erp/devel/pi
Changeset: e96d2416282f08afd7c412c182b762dc287d3726
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Wed May 24 13:25:52 2017 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/e96d2416282f08afd7c412c182b762dc287d3726 [^]

Fixes issue 34513: CreateRegFactAcct is locked just at JVM level

Removed synchronized keyword in processButton, processButtonReg and
processButtonClose methods to allow execution of process in parallel by
different threads in same JVM.

---
M src/org/openbravo/erpCommon/ad_actionButton/CreateRegFactAcct.java
---
(0096747)
aferraz   
2017-05-24 10:13   
Code review OK
(0096982)
hudsonbot   
2017-05-29 17:43   
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/1ee70113bdc4 [^]
Maturity status: Test