[Groonga-commit] droonga/fluent-plugin-droonga at 6052d07 [master] Don't add options for merging if not needed

Back to archive index

YUKI Hiroshi null+****@clear*****
Wed Dec 4 12:17:51 JST 2013


YUKI Hiroshi	2013-12-04 12:17:51 +0900 (Wed, 04 Dec 2013)

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

  Message:
    Don't add options for merging if not needed

  Modified files:
    lib/droonga/plugin/distributor/search.rb

  Modified: lib/droonga/plugin/distributor/search.rb (+4 -3)
===================================================================
--- lib/droonga/plugin/distributor/search.rb    2013-12-04 12:15:09 +0900 (c7a8e5c)
+++ lib/droonga/plugin/distributor/search.rb    2013-12-04 12:17:51 +0900 (72d2cb3)
@@ -274,12 +274,13 @@ module Droonga
         }
       end
 
-      {
+      reducer = {
         "type" => "sort",
         "operators" => operators,
-        "key_column" => key_column_index,
-        "merge_columns" => merge_columns,
       }
+      reducer["key_column"] = key_column_index unless key_column_index.nil?
+      reducer["merge_columns"] = merge_columns unless merge_columns.empty?
+      reducer
     end
   end
 end
-------------- next part --------------
HTML����������������������������...
Download 



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