[Groonga-commit] long-long-float/droonga-engine at 6651b36 [master] Use more meaningful name for "event_for_me?"

Back to archive index

YUKI Hiroshi null+****@clear*****
Thu Aug 28 16:43:57 JST 2014


YUKI Hiroshi	2014-08-28 16:43:57 +0900 (Thu, 28 Aug 2014)

  New Revision: 6651b3602c3aa1e380197cfbeef1f57a8a7f479a
  https://github.com/long-long-float/droonga-engine/commit/6651b3602c3aa1e380197cfbeef1f57a8a7f479a

  Merged 0c35c17: Merge pull request #16 from kshoji/patch-1

  Message:
    Use more meaningful name for "event_for_me?"

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

  Modified: lib/droonga/command/serf_event_handler.rb (+10 -5)
===================================================================
--- lib/droonga/command/serf_event_handler.rb    2014-08-28 16:41:14 +0900 (30afb73)
+++ lib/droonga/command/serf_event_handler.rb    2014-08-28 16:43:57 +0900 (abe023b)
@@ -44,7 +44,7 @@ module Droonga
 
       def run
         parse_event
-        unless event_for_me?
+        unless should_process?
           log(" => ignoring event not for me")
           output_response
           return true
@@ -74,11 +74,16 @@ module Droonga
         end
       end
 
-      def event_for_me?
-        return true unless @payload
-        return true unless @payload["node"]
+      def target_node
+        @payload && @payload["node"]
+      end
+
+      def for_me?
+        target_node == @serf_name
+      end
 
-        @payload["node"] == @serf_name
+      def should_process?
+        for_me? or****@paylo*****? or not****@paylo*****?("node")
       end
 
       def process_event
-------------- next part --------------
HTML����������������������������...
Download 



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