github / searchkick

Download as
Browse Code ยป

Commit 1bd814e825756a562631db740335d0b07e1e8a1b

Authored by Andrew Kane
1 parent b9a903c0
Exists in master and in 21 other branches allow_missing, conversions_period, conversions_v2, eager_load, find_in_batches, highlighted, index_name, lazy_index_name, less_deps, nested, reindex_async_threads, reindex_refactor, relation, relation_search, routing_queue, scope_attributes, search_as_you_type, synonyms_ci, test_language, threads, wildcard

Added search timeout to payload - cc @elguapo1611

Showing 3 changed files with 15 additions and 0 deletions   Show diff stats
CHANGELOG.md
  Diff comments   View file @1bd814e
  1 +## 1.3.4 [unreleased]
  2 +
  3 +- Added search timeout to payload
  4 +
1 5 ## 1.3.3
2 6  
3 7 - Fix for namespaced models (broken in 1.3.2)
... ...
lib/searchkick/query.rb
  Diff comments   View file @1bd814e
... ... @@ -368,6 +368,9 @@ module Searchkick
368 368 # highlight
369 369 set_highlights(payload, fields) if options[:highlight]
370 370  
  371 + # timeout shortly after client times out
  372 + payload[:timeout] ||= "#{Searchkick.search_timeout + 1}s"
  373 +
371 374 # An empty array will cause only the _id and _type for each hit to be returned
372 375 # doc for :select - http://www.elasticsearch.org/guide/reference/api/search/fields/
373 376 # doc for :select_v2 - https://www.elastic.co/guide/en/elasticsearch/reference/current/search-request-source-filtering.html
... ...
test/query_test.rb
  Diff comments   View file @1bd814e
... ... @@ -20,4 +20,12 @@ class QueryTest < Minitest::Test
20 20 store_names ["Milk", "Milk2"]
21 21 assert_equal ["Milk", "Milk2"], Product.search("Milk", body_options: { min_score: 0.0001 }).map(&:name)
22 22 end
  23 +
  24 + def test_default_timeout
  25 + assert_equal "6s", Product.search("*", execute: false).body[:timeout]
  26 + end
  27 +
  28 + def test_timeout_override
  29 + assert_equal "1s", Product.search("*", body_options: {timeout: "1s"}, execute: false).body[:timeout]
  30 + end
23 31 end
... ...