[Groonga-commit] groonga/groonga-query-log at 285f68d [master] Extract code that checks a command

Back to archive index

Kouhei Sutou null+****@clear*****
Fri Sep 20 12:24:39 JST 2013


Kouhei Sutou	2013-09-20 12:24:39 +0900 (Fri, 20 Sep 2013)

  New Revision: 285f68d83cf6167ef0cb3243199645b3357365ba
  https://github.com/groonga/groonga-query-log/commit/285f68d83cf6167ef0cb3243199645b3357365ba

  Message:
    Extract code that checks a command

  Modified files:
    lib/groonga/query-log/memory-leak-detector.rb

  Modified: lib/groonga/query-log/memory-leak-detector.rb (+20 -16)
===================================================================
--- lib/groonga/query-log/memory-leak-detector.rb    2013-09-20 12:23:29 +0900 (454f904)
+++ lib/groonga/query-log/memory-leak-detector.rb    2013-09-20 12:24:39 +0900 (5e7990e)
@@ -31,6 +31,26 @@ module Groonga
         each_command(input) do |command|
           command["cache"] = "no"
           @options.create_client do |client|
+            check_command(client, command)
+          rescue Groonga::Client::Connection::Error
+            # TODO: add error log mechanism
+            $stderr.puts(Time.now.iso8601)
+            $stderr.puts(statistic.command.original_source)
+            $stderr.puts($!.raw_error.message)
+            $stderr.puts($!.raw_error.backtrace)
+          end
+        end
+      end
+
+      private
+      def each_command(input)
+        parser = Parser.new
+        parser.parse(input) do |statistic|
+          yield(statistic.command)
+        end
+      end
+
+      def check_command(client, command)
             previous_memory_usage = nil
             @options.n_tries.times do |i|
               client.execute(command)
@@ -49,22 +69,6 @@ module Groonga
                 puts(command.original_source)
               end
             end
-          rescue Groonga::Client::Connection::Error
-            # TODO: add error log mechanism
-            $stderr.puts(Time.now.iso8601)
-            $stderr.puts(statistic.command.original_source)
-            $stderr.puts($!.raw_error.message)
-            $stderr.puts($!.raw_error.backtrace)
-          end
-        end
-      end
-
-      private
-      def each_command(input)
-        parser = Parser.new
-        parser.parse(input) do |statistic|
-          yield(statistic.command)
-        end
       end
 
       def memory_usage
-------------- next part --------------
HTML����������������������������...
Download 



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