Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0040017 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
backport | [Openbravo ERP] A. Platform | minor | have not tried | 2019-01-23 12:36 | 2019-01-25 12:30 | |||
Reporter | caristu | View Status | public | |||||
Assigned To | caristu | |||||||
Priority | high | Resolution | fixed | Fixed in Version | 3.0PR18Q3.5 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 042161b9b6e6 | ||||
Projection | none | ETA | none | Target Version | 3.0PR18Q3.5 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | alostale | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0040017: Concurrency problems in MyOpenbravoActionHandler | |||||||
Description | The MyOpenbravoActionHandler execution can fail if it is accessed by several threads at the same time. This is caused by [1]. As part of the changes done to decrease the time spent to load the widgets after login, the definition of the WidgetProviders is cached[2]. This kind of classes has a private field named widgetClass (a DAL object) which can cause exceptions when accessing to some of its properties in a concurrent way (See error.txt attached). [1] https://code.openbravo.com/erp/devel/pi/rev/2a1a09ee6e6852cb977efa1a987e27d2811e9127 [^] [2] https://code.openbravo.com/erp/devel/pi/rev/2a1a09ee6e6852cb977efa1a987e27d2811e9127#l1.178 [^] | |||||||
Steps To Reproduce | 1) Start Tomcat 2) Run the attached MyOpenbravoActionHandlerConcurrencyTest.test that performs several requests to MyOpenbravoActionHandler in parallel 3) The test fails because not all the requests are executed successfully | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||
|
Notes | |
(0109153) hgbot (developer) 2019-01-24 11:24 |
Repository: erp/backports/3.0PR18Q3.5 Changeset: f869a80044fec8ce9a4f8a7baa86754905c2c89f Author: Carlos Aristu <carlos.aristu <at> openbravo.com> Date: Thu Jan 24 11:23:40 2019 +0100 URL: http://code.openbravo.com/erp/backports/3.0PR18Q3.5/rev/f869a80044fec8ce9a4f8a7baa86754905c2c89f [^] related to issue 40017: apply new Java formatting --- M modules/org.openbravo.client.myob/src/org/openbravo/client/myob/WidgetProvider.java --- |
(0109154) hgbot (developer) 2019-01-24 11:24 |
Repository: erp/backports/3.0PR18Q3.5 Changeset: 042161b9b6e66cdc38d37aa2dac9d9440ee0775a Author: Carlos Aristu <carlos.aristu <at> openbravo.com> Date: Thu Jan 24 11:10:47 2019 +0100 URL: http://code.openbravo.com/erp/backports/3.0PR18Q3.5/rev/042161b9b6e66cdc38d37aa2dac9d9440ee0775a [^] fixes issue 40017: Concurrency problems in MyOpenbravoActionHandler As part of the changes done to decrease the time spent to load the widgets after login, the WidgetProviders are cached[1]. This kind of classes had a private field named widgetClass (a DAL object) which could cause errors when accessing to some of its properties in a concurrent way. To fix this issue we are now keeping in cache the properties of the WidgetClass instead of keeping the reference to the DAL object. [1] https://code.openbravo.com/erp/devel/pi/rev/2a1a09ee6e6852cb977efa1a987e27d2811e9127#l1.178 [^] --- M modules/org.openbravo.client.myob/src/org/openbravo/client/myob/WidgetProvider.java --- |
(0109251) alostale (manager) 2019-01-25 12:30 |
reviewed + tested |
Issue History | |||
Date Modified | Username | Field | Change |
2019-01-23 17:33 | caristu | Type | defect => backport |
2019-01-23 17:33 | caristu | Target Version | => 3.0PR18Q3.5 |
2019-01-24 11:24 | hgbot | Checkin | |
2019-01-24 11:24 | hgbot | Note Added: 0109153 | |
2019-01-24 11:24 | hgbot | Checkin | |
2019-01-24 11:24 | hgbot | Note Added: 0109154 | |
2019-01-24 11:24 | hgbot | Status | scheduled => resolved |
2019-01-24 11:24 | hgbot | Resolution | open => fixed |
2019-01-24 11:24 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/backports/3.0PR18Q3.5/rev/042161b9b6e66cdc38d37aa2dac9d9440ee0775a [^] |
2019-01-25 08:15 | caristu | Review Assigned To | => alostale |
2019-01-25 12:30 | alostale | Note Added: 0109251 | |
2019-01-25 12:30 | alostale | Status | resolved => closed |
2019-01-25 12:30 | alostale | Fixed in Version | => 3.0PR18Q3.5 |
Copyright © 2000 - 2009 MantisBT Group |