You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by sc...@apache.org on 2019/07/01 14:12:31 UTC

svn commit: r1862383 - /uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/resource/impl/ResourceManager_impl.java

Author: schor
Date: Mon Jul  1 14:12:30 2019
New Revision: 1862383

URL: http://svn.apache.org/viewvc?rev=1862383&view=rev
Log:
[UIMA-5935] add a check to disallow operation after destroy()

Modified:
    uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/resource/impl/ResourceManager_impl.java

Modified: uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/resource/impl/ResourceManager_impl.java
URL: http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/resource/impl/ResourceManager_impl.java?rev=1862383&r1=1862382&r2=1862383&view=diff
==============================================================================
--- uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/resource/impl/ResourceManager_impl.java (original)
+++ uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/resource/impl/ResourceManager_impl.java Mon Jul  1 14:12:30 2019
@@ -467,6 +467,7 @@ public class ResourceManager_impl implem
   @Override
   @SuppressWarnings("unchecked")
   public Class<?> getResourceClass(String aName) {
+    checkDestroyed();
     Object r = mResourceMap.get(aName);
     if (r == null) // no such resource
     {
@@ -898,6 +899,12 @@ public class ResourceManager_impl implem
       }
     }
 
+    // not clearing mResourcMap, mInternalResourceRegistrationMap, mParameterizedResourceImplClassMap, 
+    //   mInternalParameterizedResourceImplClassMap, mParameterizedResourceInstanceMap
+    //   because these could be shared with other resource managers
+    // not clearing importCache, importUrlsCache - might be in used by other Resource Managers (shared)    
+    
+    
     // no destroy of caspool at this time
     
   }