Merge pull request #1602 from shayhatsor/master

remove redundant lock in AzureGatewayListProvider
dotnet · Mar 23, 2016 · dfadfd4 · dfadfd4
2 parents c427526 + 40d3451
commit dfadfd4
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/OrleansAzureUtils/Storage/AzureGatewayListProvider.cs b/src/OrleansAzureUtils/Storage/AzureGatewayListProvider.cs
@@ -13,7 +13,6 @@ internal class AzureGatewayListProvider : IGatewayListProvider
     {
         private OrleansSiloInstanceManager siloInstanceManager;
         private ClientConfiguration config;
-        private readonly object lockable = new object();
 
         #region Implementation of IGatewayListProvider
 
@@ -25,11 +24,8 @@ public async Task InitializeGatewayListProvider(ClientConfiguration conf, TraceL
         // no caching
         public Task<IList<Uri>> GetGateways()
         {
-            lock (lockable)
-            {
-                // FindAllGatewayProxyEndpoints already returns a deep copied List<Uri>.
-                return siloInstanceManager.FindAllGatewayProxyEndpoints();
-            }
+            // FindAllGatewayProxyEndpoints already returns a deep copied List<Uri>.
+            return siloInstanceManager.FindAllGatewayProxyEndpoints();
         }
 
         public TimeSpan MaxStaleness