[Groonga-commit] droonga/droonga-engine at 83eda6b [master] Prepare codes to accept allowUpdate option

Back to archive index

YUKI Hiroshi null+****@clear*****
Tue Oct 7 12:44:27 JST 2014


YUKI Hiroshi	2014-10-07 12:44:27 +0900 (Tue, 07 Oct 2014)

  New Revision: 83eda6b76dd43129f0e5c2a1eece2cc2dd64600f
  https://github.com/droonga/droonga-engine/commit/83eda6b76dd43129f0e5c2a1eece2cc2dd64600f

  Message:
    Prepare codes to accept allowUpdate option

  Modified files:
    lib/droonga/searcher.rb

  Modified: lib/droonga/searcher.rb (+8 -0)
===================================================================
--- lib/droonga/searcher.rb    2014-10-07 12:44:04 +0900 (3cea166)
+++ lib/droonga/searcher.rb    2014-10-07 12:44:27 +0900 (3708001)
@@ -267,6 +267,7 @@ module Droonga
           options[:syntax] = :query
           options[:allow_pragma] = true
           options[:allow_column] = true
+          options[:allow_update] = false
           options[:allow_leading_not] = false
           if condition["defaultOperator"]
             default_operator_string = condition["defaultOperator"]
@@ -282,6 +283,13 @@ module Droonga
           unless condition["allowColumn"].nil?
             options[:allow_column] = condition["allowColumn"] ? true : false
           end
+          #XXX Currently we disallow to set this option to true, because
+          #    a `search` command is handled only on one replica and ignored by other replicas.
+          #    If we simply allow allowUpdate=true, replicas can be un-synchronized.
+          #    We have to do something about this problem.
+          # unless condition["allowUpdate"].nil?
+          #   options[:allow_update] = condition["allowUpdate"] ? true : false
+          # end
           unless condition["allowLeadingNot"].nil?
             options[:allow_leading_not] = condition["allowLeadingNot"] ? true : false
           end
-------------- next part --------------
HTML����������������������������...
Descargar 



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