[Groonga-commit] ranguba/rroonga at 0b46892 [master] Use all_records() for empty conditions

Zurück zum Archiv-Index

Kouhei Sutou null+****@clear*****
Wed Dec 30 12:32:49 JST 2015


Kouhei Sutou	2015-12-30 12:32:49 +0900 (Wed, 30 Dec 2015)

  New Revision: 0b4689283691ac8b3d0c63a1317788e08768d49c
  https://github.com/ranguba/rroonga/commit/0b4689283691ac8b3d0c63a1317788e08768d49c

  Message:
    Use all_records() for empty conditions
    
    Because it's faster than evaluating 'true'.

  Modified files:
    lib/groonga/expression-builder.rb

  Modified: lib/groonga/expression-builder.rb (+2 -1)
===================================================================
--- lib/groonga/expression-builder.rb    2015-11-17 22:19:29 +0900 (9db6fc9)
+++ lib/groonga/expression-builder.rb    2015-12-30 12:32:49 +0900 (191c57c)
@@ -81,7 +81,8 @@ module Groonga
       end
 
       if builders.empty?
-        expression.append_constant(true)
+        expression.append_object(@table.context["all_records"])
+        expression.append_operation(Operation::CALL, 0)
       else
         combined_builder = builders.inject(nil) do |previous, builder|
           if previous.nil?
-------------- next part --------------
HTML����������������������������...
Download 



More information about the Groonga-commit mailing list
Zurück zum Archiv-Index