[Groonga-commit] droonga/droonga-engine at 61bfea2 [master] Don't apply changes if they are not needed

Back to archive index

YUKI Hiroshi null+****@clear*****
Tue Apr 21 18:32:00 JST 2015


YUKI Hiroshi	2015-04-21 18:32:00 +0900 (Tue, 21 Apr 2015)

  New Revision: 61bfea206c40095dccb2e5ae3623587499e96ab0
  https://github.com/droonga/droonga-engine/commit/61bfea206c40095dccb2e5ae3623587499e96ab0

  Message:
    Don't apply changes if they are not needed

  Modified files:
    bin/droonga-engine-join

  Modified: bin/droonga-engine-join (+6 -6)
===================================================================
--- bin/droonga-engine-join    2015-04-21 18:20:37 +0900 (aed7ecf)
+++ bin/droonga-engine-join    2015-04-21 18:32:00 +0900 (643eb7f)
@@ -241,6 +241,7 @@ module Droonga
     end
 
     def set_source_node_role
+      return if source_node_serf.role == NodeRole::ABSORB_SOURCE
       if absorber.source_node_suspendable?
         puts("Changing role of the source node...")
         source_node_serf.ensure_restarted do
@@ -249,20 +250,20 @@ module Droonga
                                       "role" => NodeRole::ABSORB_SOURCE)
         end
       end
-      @source_node_role_changed = true
     end
 
     def set_joining_node_role
+      return if joining_node_serf.role == NodeRole::ABSORB_DESTINATION
       puts("Changing role of the joining node...")
       joining_node_serf.ensure_restarted do
         joining_node_serf.send_query("change_role",
                                      "node" => joining_node.to_s,
                                      "role" => NodeRole::ABSORB_DESTINATION)
       end
-      @joining_node_role_changed = true
     end
 
     def reset_source_node_role
+      return if source_node_serf.role == NodeRole::SERVICE_PROVIDER
       if absorber.source_node_suspendable?
         puts("Restoring role of the source node...")
         source_node_serf.ensure_restarted do
@@ -271,17 +272,16 @@ module Droonga
                                       "role" => NodeRole::SERVICE_PROVIDER)
         end
       end
-      @source_node_role_changed = false
     end
 
     def reset_joining_node_role
+      return if joining_node_serf.role == NodeRole::SERVICE_PROVIDER
       puts("Restoring role of the joining node...")
       joining_node_serf.ensure_restarted do
         joining_node_serf.send_query("change_role",
                                      "node" => joining_node.to_s,
                                      "role" => NodeRole::SERVICE_PROVIDER)
       end
-      @joining_node_role_changed = false
     end
 
     def do_join
@@ -375,8 +375,8 @@ module Droonga
     def do_cancel
       #XXX we have to write more codes to cancel remote processes!
       unregister_from_existing_nodes if @node_registered
-      reset_joining_node_role if @joining_node_role_changed
-      reset_source_node_role if @source_node_role_changed
+      reset_joining_node_role
+      reset_source_node_role
     end
     end
   end
-------------- next part --------------
HTML����������������������������...
Descargar 



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