You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@aries.apache.org by no...@apache.org on 2009/11/25 18:44:35 UTC
svn commit: r884217 -
/incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/impl/ApplicationMetadataManagerServiceImpl.java
Author: not
Date: Wed Nov 25 17:44:35 2009
New Revision: 884217
URL: http://svn.apache.org/viewvc?rev=884217&view=rev
Log:
ARIES-52 update to make the service thread safe
Modified:
incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/impl/ApplicationMetadataManagerServiceImpl.java
Modified: incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/impl/ApplicationMetadataManagerServiceImpl.java
URL: http://svn.apache.org/viewvc/incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/impl/ApplicationMetadataManagerServiceImpl.java?rev=884217&r1=884216&r2=884217&view=diff
==============================================================================
--- incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/impl/ApplicationMetadataManagerServiceImpl.java (original)
+++ incubator/aries/trunk/application/application-utils/src/main/java/org/apache/aries/application/impl/ApplicationMetadataManagerServiceImpl.java Wed Nov 25 17:44:35 2009
@@ -68,7 +68,9 @@
public boolean registerApplication(ApplicationMetadata app)
{
if (manager.registerApplication(app)) {
- appMetaData.add(app);
+ synchronized (appMetaData) {
+ appMetaData.add(app);
+ }
return true;
}
return false;
@@ -80,10 +82,12 @@
*/
public void close()
{
- for (ApplicationMetadata app : appMetaData) {
- manager.removeApplication(app);
+ synchronized (appMetaData) {
+ for (ApplicationMetadata app : appMetaData) {
+ manager.removeApplication(app);
+ }
+
+ appMetaData.clear();
}
-
- appMetaData.clear();
}
}
\ No newline at end of file