You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@brooklyn.apache.org by he...@apache.org on 2016/04/01 16:47:44 UTC

brooklyn-library git commit: ControlledDynamicWebAppCluster: improve synchronization

Repository: brooklyn-library
Updated Branches:
  refs/heads/0.9.0 76c0c720e -> 2565e6eb2


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/2565e6eb
Tree: http://git-wip-us.apache.org/repos/asf/brooklyn-library/tree/2565e6eb
Diff: http://git-wip-us.apache.org/repos/asf/brooklyn-library/diff/2565e6eb

Branch: refs/heads/0.9.0
Commit: 2565e6eb2868468ec2528df74fe85efdb887b6d2
Parents: 76c0c72
Author: Aled Sage <al...@gmail.com>
Authored: Thu Mar 31 20:48:53 2016 +0100
Committer: Alex Heneveld <al...@cloudsoftcorp.com>
Committed: Fri Apr 1 15:47:39 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/2565e6eb/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);
     }