Project:
| View Revisions: Issue #35767 | [ All Revisions ] [ Back to Issue ] | ||
| Summary | 0035767: Review synchronized methods in mobile.core, some of them may not be needed | ||
| Revision | 2017-05-15 09:12 by AugustoMauch | ||
| Description | The fix of this issue [1] removed a synchronized clause from a method to get rid of the contention is was being causing. There are other synchronized methods that should be reviewed, maybe some of them are not needed anymore. For instance the synchronized clause of this method [2] would not be needed if the attribute was initialized eagerly. Note that we must support a clustered environment, so a synchronized block cannot guarantee that a method is not executed concurrently in different application instances. [1] https://issues.openbravo.com/view.php?id=35754 [^] [2] https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/c0b32b2cd2e2/src/org/openbravo/mobile/core/servercontroller/MobileServerRequestExecutor.java#l214 [^] |
||
| Revision | 2017-05-08 09:11 by AugustoMauch | ||
| Description | The fix of this issue [1] removed a synchronized clause from a method to get rid of the contention is was being causing. There are other synchronized methods that should be reviewed, maybe some of them are not needed anymore. For instance the synchronized clause of this method [2] would not be needed if the attribute was initialized eagerly. Note that we must support a clustered environment, so we a synchronized block cannot guarantee that a method is not executed concurrently in different application instances. [1] https://issues.openbravo.com/view.php?id=35754 [^] [2] https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/c0b32b2cd2e2/src/org/openbravo/mobile/core/servercontroller/MobileServerRequestExecutor.java#l214 [^] |
||
| Copyright © 2000 - 2009 MantisBT Group |
 |