Fixed marklogic-community#456: made restart full cluster default, is …

…safer
grtjn · Jul 6, 2015 · b09931a · b09931a
1 parent 0d9e53e
commit b09931a
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/deploy/lib/server_config.rb b/deploy/lib/server_config.rb
@@ -509,18 +509,23 @@ def execute_query(query, properties = {})
   end
 
   def restart
-    group = ARGV.shift
-    # Exclude any argument passed from command line.
-    if group && group.index("-") == 0
-      group = nil
+    group = nil
+    ARGV.each do |arg|
+      # Exclude any argument passed from command line.
+      if ! arg.match("^-")
+        group = arg
+      end
     end
 
     if group && group == "cluster"
       logger.info "Restarting MarkLogic Server cluster of #{@hostname}"
     elsif group
       logger.info "Restarting MarkLogic Server group #{group}"
     else
-      logger.info "Restarting MarkLogic Server group of #{@hostname}"
+      # restarting partial cluster unsafe when working with multiple groups
+      #logger.info "Restarting MarkLogic Server group of #{@hostname}"
+      logger.info "Restarting MarkLogic Server cluster of #{@hostname}"
+      group = "cluster"
     end
     logger.debug "this: #{self}"
     setup = File.read ServerConfig.expand_path("#{@@path}/lib/xquery/setup.xqy")