[Swift-commit] cog r3580

[swift at ci.uchicago.edu]

------------------------------------------------------------------------
r3580 | hategan | 2013-02-02 02:49:11 -0600 (Sat, 02 Feb 2013) | 1 line

adding a security context there causes the stored service to not equal the updated service any more, which effectively disables the caching but also causes the cache map to grow indefinitely
------------------------------------------------------------------------
Index: modules/abstraction-common/src/org/globus/cog/abstraction/impl/file/FileResourceCache.java
===================================================================
--- modules/abstraction-common/src/org/globus/cog/abstraction/impl/file/FileResourceCache.java	(revision 3579)
+++ modules/abstraction-common/src/org/globus/cog/abstraction/impl/file/FileResourceCache.java	(working copy)
@@ -53,7 +53,7 @@
     private int maxIdleResources = DEFAULT_MAX_IDLE_RESOURCES;
     private long maxIdleTime = DEFAULT_MAX_IDLE_TIME;
     private ResourceStopper stopper;
-
+    
     public FileResourceCache() {
         fileResources = new HashMap<Service, List<FileResource>>();
         services = new HashMap<FileResource, Service>();
@@ -113,9 +113,7 @@
         if (provider == null) {
             throw new InvalidProviderException("Provider is null");
         }
-        if (service.getSecurityContext() == null) {
-            service.setSecurityContext(AbstractionFactory.getSecurityContext(provider, service.getServiceContact()));
-        }
+        
         FileResource fileResource = AbstractionFactory
                 .newFileResource(provider);
         fileResource.setService(service);