[Groonga-commit] groonga/groonga-query-log at 1c32722 [master] Revert "Add support for drilldowns.sort"

Back to archive index
Kouhei Sutou null+****@clear*****
Mon May 13 17:55:36 JST 2019


Kouhei Sutou	2019-05-13 17:55:36 +0900 (Mon, 13 May 2019)

  Revision: 1c32722f70ffad669108b29e3e0dbdf504f57bba
  https://github.com/groonga/groonga-query-log/commit/1c32722f70ffad669108b29e3e0dbdf504f57bba

  Message:
    Revert "Add support for drilldowns.sort"
    
    This reverts commit 5379dd7f43a001e23055bcdb0a9e75240765762a.
    
    This is needless.

  Modified files:
    lib/groonga-query-log/parser.rb
    test/test-parser.rb

  Modified: lib/groonga-query-log/parser.rb (+1 -1)
===================================================================
--- lib/groonga-query-log/parser.rb    2019-05-13 16:20:16 +0900 (1f0a999)
+++ lib/groonga-query-log/parser.rb    2019-05-13 17:55:36 +0900 (c02ce44)
@@ -121,7 +121,7 @@ module GroongaQueryLog
                        \ 
                        (?<raw_message>
                          (?<name>[a-zA-Z._-]+)
-                         (?<sub_name_before>\[.+?\][^(]*)?
+                         (?<sub_name_before>\[.+?\](?:[a-zA-Z._-]*))?
                          (?:\((?<n_records>\d+)\))?
                          (?<sub_name_after>\[.+?\])?
                          (?::\ (?<extra>.*))?

  Modified: test/test-parser.rb (+0 -31)
===================================================================
--- test/test-parser.rb    2019-05-13 16:20:16 +0900 (37789c4)
+++ test/test-parser.rb    2019-05-13 17:55:36 +0900 (a581b98)
@@ -326,37 +326,6 @@ class ParserTest < Test::Unit::TestCase
       ]
       assert_equal(expected, operations)
     end
-
-    def test_labeled_drilldown_sort
-      statistics = parse(<<-LOG)
-2019-05-13 16:18:34.272078|0x7ffcb639c7f0|>select Memos   --filter true   --drilldowns[tag].keys tag,date   --drilldowns[tag].output_columns _key[0],_key[1],_nsubrecs   --drilldowns[tag].sort_keys _value.tag._key,_value.date --output_type json
-2019-05-13 16:18:34.297587|0x7ffcb639c7f0|:000000025513058 filter(5): true
-2019-05-13 16:18:34.297595|0x7ffcb639c7f0|:000000025518645 select(5)
-2019-05-13 16:18:34.297657|0x7ffcb639c7f0|:000000025581524 drilldowns[tag](4)
-2019-05-13 16:18:34.297756|0x7ffcb639c7f0|:000000025680289 output(5)
-2019-05-13 16:18:34.297828|0x7ffcb639c7f0|:000000025751970 drilldowns[tag].sort[_value.tag._key,_value.date](4)
-2019-05-13 16:18:34.297911|0x7ffcb639c7f0|:000000025835741 output.drilldowns[tag](4)
-2019-05-13 16:18:34.297980|0x7ffcb639c7f0|<000000025904314 rc=0
-      LOG
-      operations = statistics.first.operations.collect do |operation|
-        [operation[:name], operation[:raw_message]]
-      end
-      expected = [
-        ["filter",
-         "filter(5): true"],
-        ["select",
-         "select(5)"],
-        ["drilldowns[tag]",
-         "drilldowns[tag](4)"],
-        ["output",
-         "output(5)"],
-        ["drilldowns[tag].sort[_value.tag._key,_value.date]",
-         "drilldowns[tag].sort[_value.tag._key,_value.date](4)"],
-        ["output.drilldowns[tag]",
-         "output.drilldowns[tag](4)"],
-      ]
-      assert_equal(expected, operations)
-    end
   end
 
   class ExtraFieldTest < self
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.osdn.me/mailman/archives/groonga-commit/attachments/20190513/5231f47f/attachment-0001.html>


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