[Groonga-commit] groonga/groonga at 236fd93 [master] logical_select: fix a bug that wrong cache is used

Back to archive index

Kouhei Sutou null+****@clear*****
Wed Mar 29 16:42:38 JST 2017


Kouhei Sutou	2017-03-29 16:42:38 +0900 (Wed, 29 Mar 2017)

  New Revision: 236fd934907cb93a6e8e1acf371d44d0cf4111f0
  https://github.com/groonga/groonga/commit/236fd934907cb93a6e8e1acf371d44d0cf4111f0

  Message:
    logical_select: fix a bug that wrong cache is used
    
    Dynamic column parameters aren't included in cache key.

  Modified files:
    plugins/sharding/logical_select.rb

  Modified: plugins/sharding/logical_select.rb (+4 -2)
===================================================================
--- plugins/sharding/logical_select.rb    2017-03-29 16:42:26 +0900 (85d3fd8)
+++ plugins/sharding/logical_select.rb    2017-03-29 16:42:38 +0900 (262732a)
@@ -110,13 +110,15 @@ module Groonga
           :filtered,
           :output
         ].each do |stage|
-          target_dynamic_columns = dynamic_columns.__send__("each_#{stage}")
+          target_dynamic_columns = dynamic_columns.__send__("each_#{stage}").to_a
           target_dynamic_columns.sort_by(&:label).each do |dynamic_column|
             key << "#{dynamic_column.label}\0"
             key << "#{dynamic_column.stage}\0"
             key << "#{dynamic_column.type}\0"
-            key << "#{dynamic_column.flags.join('|')}\0"
+            key << "#{dynamic_column.flags}\0"
             key << "#{dynamic_column.value}\0"
+            key << "#{dynamic_column.window_sort_keys.join(',')}\0"
+            key << "#{dynamic_column.window_group_keys.join(',')}\0"
           end
         end
       end
-------------- next part --------------
HTML����������������������������...
다운로드 



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