inquisitio 1.2.1 → 1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ec481031a9aa085e8b4a41f8cfa054415645f7e1
-  data.tar.gz: 3c52019c27f4b86f38a2da70dd063ec58b2be998
+  metadata.gz: 34e8488ee37d4001d57ec5ed0a318b2532d47f2f
+  data.tar.gz: 1a20339ae3942c0e5ba5c0828b725b5f8039f6e0
 SHA512:
-  metadata.gz: 289d6a4cf32a0a112648c82e9ad38f755eac8a30dc8e13f804d3a01af58f63543843c73f1d2fb4c45645e5a5808fc43ab67fcef426edc5047db8d783f6bb52cd
-  data.tar.gz: ce0514c3be78138eb48447b5cf098025d437bb4831dfe408df120e3cb6a5b86d2a2437dae078cb5b6c2e10ef3b7a99d672ccb9f85fcaebed8441b4fbc2ac1101
+  metadata.gz: 37b7902705751a607286325de49dd77f331e2ce1c7dd4db4115f09ec40553f88c65f66cde534cbec9691982992a7ede7609b1c613761344857e702cadac1a03d
+  data.tar.gz: 034fe6edaa9e2cb468765eec84582e4220b4e2d22d0e37ab686e8efc211aad6ad2086f499c29564811a4896714cece5631c1f3b72d65fc58545f1f6faec7a6ee

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+1.2.2 / 2014-06-03
+[FEATURE] Return time taken by queries as time_ms on Searcher
+
 1.2.1 / 2014-06-03
 [BUGFIX] Fixed start of search offset
 

data/lib/inquisitio/indexer.rb

@@ -15,7 +15,7 @@ module Inquisitio
     end
 
     def index
-      Inquisitio.config.logger.info "Indexer posting to #{batch_index_url} body: #{body}"
+      Inquisitio.config.logger.info "Indexer posting to #{batch_index_url}"
       if Inquisitio.config.dry_run
         Inquisitio.config.logger.info "Skipping POST as running in dry-run mode"
       else
@@ -26,8 +26,7 @@ module Inquisitio
     private
 
     def body
-      body = @documents.map(&:to_SDF).join(", ")
-      "[#{body}]"
+      @body ||= "[#{@documents.map(&:to_SDF).join(", ")}]"
     end
 
     def batch_index_url
@@ -38,7 +37,7 @@ module Inquisitio
       response = Excon.post(batch_index_url,
                            :body => body,
                            :headers => {"Content-Type" =>"application/json"})
-      Inquisitio.config.logger.info "Response - status: #{response.status} body: #{response.body}"
+      Inquisitio.config.logger.info "Response - status: #{response.status}"
       raise InquisitioError.new("Index failed with status code: #{response.status} Message: #{response.body}") unless response.status == 200
       response.body
     end    

data/lib/inquisitio/results.rb

@@ -1,13 +1,11 @@
 module Inquisitio
   class Results < Array
-    def initialize(items, current_page, results_per_page, total_count)
+    def initialize(items, current_page, results_per_page, total_count, time_ms)
       super(items)
-      @current_page     = current_page
-      @results_per_page = results_per_page
-      @total_count      = total_count
+      @current_page, @results_per_page, @total_count, @time_ms = current_page, results_per_page, total_count, time_ms
     end
 
-    attr_reader :total_count, :results_per_page, :current_page
+    attr_reader :total_count, :results_per_page, :current_page, :time_ms
     alias_method :total_entries, :total_count
     alias_method :limit_value, :results_per_page
 

data/lib/inquisitio/searcher.rb

@@ -125,10 +125,11 @@ module Inquisitio
       response = Excon.get(search_url)
       raise InquisitioError.new("Search failed with status code: #{response.status} Message #{response.body}") unless response.status == 200
       body = JSON.parse(response.body)
-      @results = Results.new(body["hits"]["hit"],
+      @results = Results.new(body['hits']['hit'],
         params[:page],
         params[:per],
-        body["hits"]["found"])
+        body['hits']['found'],
+        body['info']['time-ms'])
     rescue => e
       @failed_attempts += 1
       Inquisitio.config.logger.error("Exception Performing search: #{search_url} #{e}")

data/lib/inquisitio/version.rb

@@ -1,3 +1,3 @@
 module Inquisitio
-  VERSION = "1.2.1"
+  VERSION = "1.2.2"
 end

data/test/results_test.rb

@@ -32,6 +32,12 @@ module Inquisitio
       assert_equal @found, searcher.total_entries
     end
 
+    def test_should_return_time_taken
+      searcher = Searcher.where("star_wars")
+      searcher.search
+      assert_equal 3, searcher.time_ms
+    end
+
     def test_total_entries_should_proxy
       searcher = Searcher.where("star_wars")
       searcher.search
@@ -88,13 +94,13 @@ module Inquisitio
     end
 
     def test_last_page_before
-      results = Inquisitio::Results.new([], 1, nil, nil)
+      results = Inquisitio::Results.new([], 1, nil, nil, 7)
       results.expects(total_pages: 2)
       refute results.last_page?
     end
 
     def test_last_page_equal
-      results = Inquisitio::Results.new([], 2, nil, nil)
+      results = Inquisitio::Results.new([], 2, nil, nil, 7)
       results.expects(total_pages: 2)
       assert results.last_page?
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inquisitio
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.2
 platform: ruby
 authors:
 - Jeremy Walker
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-03 00:00:00.000000000 Z
+date: 2014-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: excon