[Groonga-commit] droonga/fluent-plugin-droonga at ecd7fcc [master] Remove needless tag and time from job data in queue

Back to archive index

Kouhei Sutou null+****@clear*****
Sun Nov 24 23:35:12 JST 2013


Kouhei Sutou	2013-11-24 23:35:12 +0900 (Sun, 24 Nov 2013)

  New Revision: ecd7fcccbbbbdb393e5b5583196432d8cf5dc8eb
  https://github.com/droonga/fluent-plugin-droonga/commit/ecd7fcccbbbbdb393e5b5583196432d8cf5dc8eb

  Message:
    Remove needless tag and time from job data in queue

  Modified files:
    lib/droonga/handler.rb

  Modified: lib/droonga/handler.rb (+8 -19)
===================================================================
--- lib/droonga/handler.rb    2013-11-24 23:31:19 +0900 (1a8c4c4)
+++ lib/droonga/handler.rb    2013-11-24 23:35:12 +0900 (0267071)
@@ -58,12 +58,12 @@ module Droonga
 
     def execute_one
       $log.trace("#{log_tag}: execute_one: start")
-      message = @job_queue.pull_message
-      unless message
+      envelope = @job_queue.pull_message
+      unless envelope
         $log.trace("#{log_tag}: execute_one: abort: no message")
         return
       end
-      process(message)
+      process(envelope)
       $log.trace("#{log_tag}: execute_one: done")
     end
 
@@ -71,9 +71,9 @@ module Droonga
       find_plugin(command).prefer_synchronous?(command)
     end
 
-    def process(message)
+    def process(envelope)
       $log.trace("#{log_tag}: process: start")
-      body, command, arguments = parse_message(message)
+      body, command, arguments = parse_envelope(envelope)
       plugin = find_plugin(command)
       if plugin.nil?
         $log.trace("#{log_tag}: process: done: no plugin: <#{command}>")
@@ -119,8 +119,7 @@ module Droonga
         envelope["body"] = body
         envelope["type"] = command
         envelope["arguments"] = arguments
-        message = ['', Time.now.to_f, envelope]
-        @job_queue.push_message(message)
+        @job_queue.push_message(envelope)
       end
     end
 
@@ -170,18 +169,8 @@ module Droonga
       $log.trace("#{log_tag}: output: done")
     end
 
-    def parse_message(message)
-      tag, time, record = message
-      prefix, type, *arguments = tag.split(/\./)
-      if type.nil? || type.empty? || type == 'message'
-        @envelope = record
-      else
-        @envelope = {
-          "type" => type,
-          "arguments" => arguments,
-          "body" => record
-        }
-      end
+    def parse_envelope(envelope)
+      @envelope = envelope
       envelope["via"] ||= []
       [envelope["body"], envelope["type"], envelope["arguments"]]
     end
-------------- next part --------------
HTML����������������������������...
Download 



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