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

[burn]

Author: burn
Date: Mon May 16 18:15:09 2016
New Revision: 1744104

URL: http://svn.apache.org/viewvc?rev=1744104&view=rev
Log:
UIMA-4928 Update meta-props when a service's autostart option is modified

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

Modified: 
uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceHandler.java
URL: 
http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceHandler.java?rev=1744104&r1=1744103&r2=1744104&view=diff
==============================================================================
--- 
uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceHandler.java
 (original)
+++ 
uima/sandbox/uima-ducc/trunk/uima-ducc-sm/src/main/java/org/apache/uima/ducc/sm/ServiceHandler.java
 Mon May 16 18:15:09 2016
@@ -1166,6 +1166,7 @@ public class ServiceHandler
         DuccProperties mods  = sme.getProperties();
         restart_pinger = false;
         restart_service = false;
+        boolean updateMeta = false;
         Set<String> keys = mods.stringPropertyNames();        
 
         for (String kk : keys ) {
@@ -1182,9 +1183,9 @@ public class ServiceHandler
                 case Modify:
                     // used by CLI only, won't even be passed in
                     continue;
-                //case Activate:
-                    // TODO: I don't think this is ever used.  Maybe just drop 
it?
-                    //continue;
+                case Autostart:
+                    updateMeta = true;           // UIMA-4928 (Should move it 
to the svc props)
+                default:
             }
 
             String v = (String) mods.get(kk);
@@ -1200,10 +1201,13 @@ public class ServiceHandler
         
         sset.resetRuntimeErrors();
         try {
-                       sset.updateSvcProperties();
-               } catch (Exception e) {
-                       logger.error(methodName, sset.getId(), "Cannot store 
properties:", e);
-               }
+            sset.updateSvcProperties();
+            if (updateMeta) {
+                sset.updateMetaProperties();
+            }
+        } catch (Exception e) {
+            logger.error(methodName, sset.getId(), "Cannot store properties:", 
e);
+        }
 
         if ( restart_pinger ) {
             sset.restartPinger();