You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by ch...@apache.org on 2015/11/02 20:15:07 UTC

svn commit: r1712117 - /uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceSet.java

Author: challngr
Date: Mon Nov  2 19:15:07 2015
New Revision: 1712117

URL: http://svn.apache.org/viewvc?rev=1712117&view=rev
Log:
UIMA-4587 Stop updating meta state after unregister.  Happens if state continues to arrive after unregister.

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceSet.java

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceSet.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceSet.java?rev=1712117&r1=1712116&r2=1712117&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceSet.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceSet.java Mon Nov  2 19:15:07 2015
@@ -842,8 +842,10 @@ public class ServiceSet
     	throws Exception
     {
     	// String methodName = "saveMetaProperties";
-        // UIMA-4587 Why bypass, as state can still dribble in.
-        // if ( isDeregistered() ) return;
+        if ( isDeregistered() ) return;                // we may have deleted the properties but stuff
+                                                       // lingers out of our control.  no more updates
+                                                       // which can leave junk in the registry directory
+                                                       // for file-based registry.
 
         prepareMetaProperties();
         stateHandler.updateMetaProperties(id, meta_props);