Openbravo Issue Tracking System - Openbravo ERP | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0030181 | Openbravo ERP | A. Platform | public | 2015-06-15 14:25 | 2022-02-01 08:05 |
Reporter | shuehner | ||||
Assigned To | Triage Platform Base | ||||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | acknowledged | Resolution | open | ||
Platform | OS | 5 | OS Version | ||
Product Version | |||||
Target Version | Fixed in Version | ||||
Merge Request Status | |||||
Review Assigned To | |||||
OBNetwork customer | |||||
Web browser | |||||
Modules | Core | ||||
Support ticket | |||||
Regression level | |||||
Regression date | |||||
Regression introduced in release | |||||
Regression introduced by commit | |||||
Triggers an Emergency Pack | No | ||||
Summary | 0030181: Every Modulescript executes creates own new connectionpool which are never properly closed | ||||
Description | EveryModuleScript does creates a entirely new ConnectionPool (via calling getConnectionProvider) which are never closed. So during exection of i.e. update.database many connection pool equal to number of ModuleScripts are created and never closed Note: end of java process will close them. However that behavior does not make any sense and is just wrong. | ||||
Steps To Reproduce | read src-core/src/org/openbravo/modulescript/ModuleScript.java source or add some debugging to i.e. constructor of ConnectionProviderImpl | ||||
Proposed Solution | One option maybe would be to make 1.) cp variable in ModuleScript static 2.) maybe close that single pool and end of 'execute' of ModuleScriptHandler That way all would share single pool and close this pool properly. Note: need to think of side-effects of that static first to check if that is a viable option. | ||||
Additional Information | |||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | |||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2015-06-15 14:25 | shuehner | New Issue | |||
2015-06-15 14:25 | shuehner | Assigned To | => platform | ||
2015-06-15 14:25 | shuehner | Modules | => Core | ||
2015-06-15 14:25 | shuehner | Triggers an Emergency Pack | => No | ||
2015-06-16 11:49 | alostale | Status | new => acknowledged | ||
2022-02-01 08:05 | alostale | Assigned To | platform => Triage Platform Base |
There are no notes attached to this issue. |