groonga-query-log 1.7.2 → 1.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b9d218a019817b6222c63db7cb6dcde54452d56dba3ad1026e678e0d64a8ca1c
-  data.tar.gz: 4755e11de2ab221a1db0a1d6c2fa86c3bf05e18c3cd30e451096160bf870e169
+  metadata.gz: fca209c185dcd27bb15bdf7444261a754f33d40f73ffe1f1141fc8fe5a2c61ac
+  data.tar.gz: 4efd951340f55639e8d83fbd69d13ef411ca6c62bc0f4bfae59afd982c9d2571
 SHA512:
-  metadata.gz: 1380025523bbb3debb587da39f774f7e2cb7aab6301e2d6f66ebf9c7fed3baccbfa5e4fc57202db3e1d00c53a37686196d1165df560ba9c824cce68efd21af03
-  data.tar.gz: ed36cd1e75dc79eb701238bad818ee0a42531d1ff47ebaba255bdfeb68574f173f49e44f0e38e786d45d41981d720c6ac2e5d32f6a84fa8abebdcde0adebe8bb
+  metadata.gz: fd72aaaf751c2acee54c88eeb9e8b04d528d5d9debc29405afbba617341b89e59012cb8e34fe2e5088b8e945aa8bf8f2610f4329a16b848c51c51ebdf4b75d3e
+  data.tar.gz: b366ad6ce14ffd22da56c7f4432cf59856a6896bdf1a13ebad9d9689043934ab3a13722fb76bb7b5a22a7cdcecc00da62e29940bf191a98b0483bc63af4e17f1

data/doc/text/news.md

@@ -1,5 +1,20 @@
 # News
 
+## 1.7.3: 2020-11-12
+
+### Improvements
+
+  * `check-crash`: Added support for showing Groonga version.
+
+  * `server-verifier`: Added  `--max-limit` option.
+
+  * `run-regression-test`: Added `--max-limit` option.
+
+### Fixes
+
+  * `GroongaQueryLog::Parser`: Fixed a bug that target command filter
+    may not work.
+
 ## 1.7.2: 2020-06-02
 
 ### Fixes

data/lib/groonga-query-log/command/check-crash.rb

@@ -71,6 +71,7 @@ module GroongaQueryLog
       end
 
       class GroongaProcess
+        attr_reader :version
         attr_reader :pid
         attr_reader :start_time
         attr_reader :start_log_path
@@ -80,7 +81,8 @@ module GroongaQueryLog
         attr_writer :crashed
         attr_writer :finished
         attr_reader :important_entries
-        def initialize(pid, start_time, start_log_path)
+        def initialize(version, pid, start_time, start_log_path)
+          @version = version
           @pid = pid
           @start_time = start_time
           @end_time = @start_time
@@ -121,6 +123,7 @@ module GroongaQueryLog
               need_query_log_parsing = false
               p [:process,
                  :success,
+                 process.version,
                  process.start_time.iso8601,
                  process.end_time.iso8601,
                  process.pid,
@@ -129,6 +132,7 @@ module GroongaQueryLog
             elsif process.crashed?
               p [:process,
                  :crashed,
+                 process.version,
                  process.start_time.iso8601,
                  process.end_time.iso8601,
                  process.pid,
@@ -137,6 +141,7 @@ module GroongaQueryLog
             else
               p [:process,
                  :unfinished,
+                 process.version,
                  process.start_time.iso8601,
                  process.pid,
                  process.start_log_path]
@@ -144,6 +149,7 @@ module GroongaQueryLog
 
             unless process.n_leaks.zero?
               p [:leak,
+                 process.version,
                  process.n_leaks,
                  process.end_time.iso8601,
                  process.pid,
@@ -184,7 +190,8 @@ module GroongaQueryLog
               end
             end
             unless @unflushed_statistics.empty?
-              puts("Unflushed commands in #{start.iso8601}/#{last.iso8601}")
+              puts("Unflushed commands in " +
+                   "#{start_time.iso8601}/#{end_time.iso8601}")
               @unflushed_statistics.each do |statistic|
                 puts("#{statistic.start_time.iso8601}: #{statistic.raw_command}")
               end
@@ -320,7 +327,8 @@ module GroongaQueryLog
           end
 
           case entry.message
-          when /\Agrn_init:/, /\Amroonga \d+\.\d+ started\.\z/
+          when /\Agrn_init: <(.+?)>/, /\Amroonga (\d+\.\d+) started\.\z/
+            version = $1
             process = @running_processes[entry.pid]
             if process
               process.finished = true
@@ -328,12 +336,15 @@ module GroongaQueryLog
               yield(process)
               @running_processes.delete(entry.pid)
             end
-            process = GroongaProcess.new(entry.pid, entry.timestamp, path)
+            process = GroongaProcess.new(version,
+                                         entry.pid,
+                                         entry.timestamp,
+                                         path)
             @running_processes[entry.pid] = process
           when /\Agrn_fin \((\d+)\)\z/
             n_leaks = $1.to_i
             @running_processes[entry.pid] ||=
-              GroongaProcess.new(entry.pid, Time.at(0), path)
+              GroongaProcess.new(nil, entry.pid, Time.at(0), path)
             process = @running_processes[entry.pid]
             process.n_leaks = n_leaks
             process.end_time = entry.timestamp
@@ -343,7 +354,7 @@ module GroongaQueryLog
             @running_processes.delete(entry.pid)
           else
             @running_processes[entry.pid] ||=
-              GroongaProcess.new(entry.pid, Time.at(0), path)
+              GroongaProcess.new(nil, entry.pid, Time.at(0), path)
             process = @running_processes[entry.pid]
             case entry.log_level
             when :notice

data/lib/groonga-query-log/command/run-regression-test.rb

@@ -66,6 +66,7 @@ module GroongaQueryLog
         @rewrite_and_not_operator = false
         @debug_rewrite = false
         @omit_rate = 0.0
+        @max_limit = -1
 
         @care_order = true
         @ignored_drilldown_keys = []
@@ -313,6 +314,12 @@ module GroongaQueryLog
                   "(#{@omit_rate})") do |rate|
           @omit_rate = rate
         end
+        parser.on("--max-limit=LIMIT",
+                  "Use LIMIT as the max limit value",
+                  "Negative value doesn't rewrite the limit parameter",
+                  "(#{@max_limit})") do |limit|
+          @max_limit = limit
+        end
 
         parser.separator("")
         parser.separator("Comparisons:")
@@ -466,6 +473,7 @@ module GroongaQueryLog
             @rewrite_and_not_operator,
           :debug_rewrite => @debug_rewrite,
           :omit_rate => @omit_rate,
+          :max_limit => @max_limit,
           :target_command_names => @target_command_names,
           :verify_performance => @verify_performance,
           :performance_verfifier_options => @performance_verfifier_options,
@@ -919,6 +927,10 @@ module GroongaQueryLog
             command_line << "--omit-rate"
             command_line << @options[:omit_rate].to_s
           end
+          if @options[:max_limit] >= 0
+            command_line << "--max-limit"
+            command_line << @options[:max_limit].to_s
+          end
           if @options[:target_command_names]
             command_line << "--target-command-names"
             command_line << @options[:target_command_names].join(",")

data/lib/groonga-query-log/command/verify-server.rb

@@ -1,4 +1,4 @@
-# Copyright (C) 2013-2018  Kouhei Sutou <kou@clear-code.com>
+# Copyright (C) 2013-2020  Sutou Kouhei <kou@clear-code.com>
 # Copyright (C) 2020  Horimoto Yasuhiro <horimoto@clear-code.com>
 #
 # This library is free software; you can redistribute it and/or
@@ -280,6 +280,13 @@ module GroongaQueryLog
           @options.nullable_reference_number_accessors << accessor
         end
 
+        parser.on("--max-limit=LIMIT",
+                  "Use LIMIT as the max limit value",
+                  "Negative value doesn't rewrite the limit parameter",
+                  "(#{@options.max_limit})") do |limit|
+          @options.max_limit = limit
+        end
+
         create_parser_performance(parser)
 
         parser.separator("")

data/lib/groonga-query-log/parser.rb

@@ -160,7 +160,7 @@ module GroongaQueryLog
 
     def target_statistic?(statistic)
       if @target_commands
-        unless @target_commands.include?(statistic.command.name)
+        unless @target_commands.include?(statistic.command.command_name)
           return false
         end
       end

data/lib/groonga-query-log/replayer.rb

@@ -83,19 +83,15 @@ module GroongaQueryLog
           end
           begin
             replay_command(client, id, statistic.command)
-          rescue Groonga::Client::Error
+          rescue => error
             # TODO: add error log mechanism
             $stderr.puts(Time.now.iso8601(6))
             $stderr.puts(statistic.command.original_source)
-            $stderr.puts($!.raw_error.message)
-            $stderr.puts($!.raw_error.backtrace)
-            return false
-          rescue
-            # TODO: add error log mechanism
-            $stderr.puts(Time.now.iso8601(6))
-            $stderr.puts(statistic.command.original_source)
-            $stderr.puts($!.message)
-            $stderr.puts($!.backtrace)
+            if error.respond_to?(:raw_error)
+              error = raw_error
+            end
+            $stderr.puts(error.message)
+            $stderr.puts(error.backtrace)
             return false
           end
         end

data/lib/groonga-query-log/server-verifier.rb

@@ -1,4 +1,4 @@
-# Copyright (C) 2013-2018  Kouhei Sutou <kou@clear-code.com>
+# Copyright (C) 2013-2020  Sutou Kouhei <kou@clear-code.com>
 # Copyright (C) 2020  Horimoto Yasuhiro <horimoto@clear-code.com>
 #
 # This library is free software; you can redistribute it and/or
@@ -170,6 +170,14 @@ module GroongaQueryLog
     def verify_command(groonga1_client, groonga2_client, command)
       command["cache"] = "no" if @options.disable_cache?
       command["cache"] = "no" if @options.verify_performance?
+      if @options.max_limit >= 0 and command["limit"]
+        limit = command["limit"].to_i
+        if limit >= 0
+          command["limit"] = [limit, @options.max_limit].min.to_s
+        else
+          command["limit"] = @options.max_limit.to_s
+        end
+      end
       command["output_type"] = "json"
       rewrite_filter(command, "filter")
       rewrite_filter(command, "scorer")
@@ -317,6 +325,7 @@ module GroongaQueryLog
       attr_reader :performance_verifier_options
       attr_writer :debug_rewrite
       attr_writer :omit_rate
+      attr_accessor :max_limit
       def initialize
         @groonga1 = GroongaOptions.new
         @groonga2 = GroongaOptions.new
@@ -351,6 +360,7 @@ module GroongaQueryLog
         @performance_verifier_options = PerformanceVerifier::Options.new
         @debug_rewrite = false
         @omit_rate = 0.0
+        @max_limit = -1
       end
 
       def request_queue_size

data/lib/groonga-query-log/version.rb

@@ -15,5 +15,5 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
 
 module GroongaQueryLog
-  VERSION = "1.7.2"
+  VERSION = "1.7.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: groonga-query-log
 version: !ruby/object:Gem::Version
-  version: 1.7.2
+  version: 1.7.3
 platform: ruby
 authors:
 - Kouhei Sutou
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-01 00:00:00.000000000 Z
+date: 2020-11-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: charty
@@ -315,7 +315,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.0.pre1
+rubygems_version: 3.2.0.rc.2
 signing_key: 
 specification_version: 4
 summary: Groonga-query-log is a collection of library and tools to process [Groonga](http://groonga.org/)'s