brooklyn-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From henev...@apache.org
Subject [1/2] brooklyn-library git commit: ControlledDynamicWebAppCluster: improve synchronization
Date Fri, 01 Apr 2016 14:46:45 GMT
Repository: brooklyn-library
Updated Branches:
  refs/heads/master 8be4ecb65 -> 6541b638f


ControlledDynamicWebAppCluster: improve synchronization

Synchronising on these getters is not necessary, because the
values are set in init().

Project: http://git-wip-us.apache.org/repos/asf/brooklyn-library/repo
Commit: http://git-wip-us.apache.org/repos/asf/brooklyn-library/commit/0e2f28cc
Tree: http://git-wip-us.apache.org/repos/asf/brooklyn-library/tree/0e2f28cc
Diff: http://git-wip-us.apache.org/repos/asf/brooklyn-library/diff/0e2f28cc

Branch: refs/heads/master
Commit: 0e2f28cc41a0dd8bee6b5b8d74e98f5d12961231
Parents: 3216b7a
Author: Aled Sage <aled.sage@gmail.com>
Authored: Thu Mar 31 20:48:53 2016 +0100
Committer: Aled Sage <aled.sage@gmail.com>
Committed: Thu Mar 31 20:48:53 2016 +0100

----------------------------------------------------------------------
 .../entity/webapp/ControlledDynamicWebAppClusterImpl.java        | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/brooklyn-library/blob/0e2f28cc/software/webapp/src/main/java/org/apache/brooklyn/entity/webapp/ControlledDynamicWebAppClusterImpl.java
----------------------------------------------------------------------
diff --git a/software/webapp/src/main/java/org/apache/brooklyn/entity/webapp/ControlledDynamicWebAppClusterImpl.java
b/software/webapp/src/main/java/org/apache/brooklyn/entity/webapp/ControlledDynamicWebAppClusterImpl.java
index 8c52746..6ac61ce 100644
--- a/software/webapp/src/main/java/org/apache/brooklyn/entity/webapp/ControlledDynamicWebAppClusterImpl.java
+++ b/software/webapp/src/main/java/org/apache/brooklyn/entity/webapp/ControlledDynamicWebAppClusterImpl.java
@@ -184,13 +184,13 @@ public class ControlledDynamicWebAppClusterImpl extends DynamicGroupImpl
impleme
 
     @SuppressWarnings("unchecked")
     @Override
-    public synchronized ConfigurableEntityFactory<WebAppService> getFactory() {
+    public ConfigurableEntityFactory<WebAppService> getFactory() {
         return (ConfigurableEntityFactory<WebAppService>) getAttribute(FACTORY);
     }
     
     // TODO convert to an entity reference which is serializable
     @Override
-    public synchronized DynamicWebAppCluster getCluster() {
+    public DynamicWebAppCluster getCluster() {
         return getAttribute(CLUSTER);
     }
     


Mime
View raw message