From 0e2f28cc41a0dd8bee6b5b8d74e98f5d12961231 Mon Sep 17 00:00:00 2001
From: Aled Sage <aled.sage@gmail.com>
Date: Thu, 31 Mar 2016 20:48:53 +0100
Subject: [PATCH] ControlledDynamicWebAppCluster: improve synchronization

Synchronising on these getters is not necessary, because the
values are set in init().
---
 .../entity/webapp/ControlledDynamicWebAppClusterImpl.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

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 8c5274671..6ac61ce4a 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 LoadBalancer getController() {
 
     @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);
     }