[Groonga-commit] droonga/droonga-engine at 6b4422a [master] Revert "Add an option to disable orchestration based on Serf."

Back to archive index

YUKI Hiroshi null+****@clear*****
Tue Oct 7 18:21:00 JST 2014


YUKI Hiroshi	2014-10-07 18:21:00 +0900 (Tue, 07 Oct 2014)

  New Revision: 6b4422a2e191edd7155d2918299d26f50895577c
  https://github.com/droonga/droonga-engine/commit/6b4422a2e191edd7155d2918299d26f50895577c

  Message:
    Revert "Add an option to disable orchestration based on Serf."
    
    This reverts commit d6d03af40892692ed2f80b7367b3c8872c4c5343.
    We should not include such a test-specific feature!

  Modified files:
    lib/droonga/command/droonga_engine.rb

  Modified: lib/droonga/command/droonga_engine.rb (+4 -14)
===================================================================
--- lib/droonga/command/droonga_engine.rb    2014-10-07 18:20:18 +0900 (55e3c12)
+++ lib/droonga/command/droonga_engine.rb    2014-10-07 18:21:00 +0900 (7e94a6c)
@@ -114,7 +114,7 @@ module Droonga
 
       class Configuration
         attr_reader :host, :port, :tag, :log_file, :pid_file_path
-        attr_reader :ready_notify_fd, :orchestrated
+        attr_reader :ready_notify_fd
         def initialize
           config = load_config
 
@@ -128,7 +128,6 @@ module Droonga
           @daemon          = false
           @pid_file_path   = nil
           @ready_notify_fd = nil
-          @orchestrated = true
 
           if have_config_file?
             if config.include?("daemon")
@@ -225,7 +224,6 @@ module Droonga
           add_log_options(parser)
           add_process_options(parser)
           add_path_options(parser)
-          add_orchestration_options(parser)
           add_notification_options(parser)
         end
 
@@ -310,13 +308,6 @@ module Droonga
           end
         end
 
-        def add_orchestration_options(parser)
-          parser.on("--no-orchestration",
-                    "Disable orchestration with other nodes") do
-            @orchestrated = false
-          end
-        end
-
         def add_notification_options(parser)
           parser.separator("")
           parser.separator("Notification:")
@@ -348,7 +339,7 @@ module Droonga
 
           trap_signals
           @loop.run
-          @serf.stop if @serf and****@serf*****?
+          @serf.stop if****@serf*****?
 
           @service_runner.success?
         end
@@ -389,14 +380,14 @@ module Droonga
 
         def stop_gracefully
           @command_runner.stop
-          @serf.stop if @serf
+          @serf.stop
           @catalog_observer.stop
           @service_runner.stop_gracefully
         end
 
         def stop_immediately
           @command_runner.stop
-          @serf.stop if @serf
+          @serf.stop
           @catalog_observer.stop
           @service_runner.stop_immediately
         end
@@ -427,7 +418,6 @@ module Droonga
         end
 
         def run_serf
-          return nil unles****@confi*****
           serf = Serf.new(@loop, @configuration.engine_name)
           serf.start
           serf
-------------- next part --------------
HTML����������������������������...
다운로드 



More information about the Groonga-commit mailing list
Back to archive index