Merge branch 'bogdanovich-master'

Andrew Kane
2 parents 06661b2e a677d983
Showing 6 changed files with 51 additions and 5 deletions Show diff stats
CHANGELOG.md
README.md
lib/searchkick/index.rb
lib/searchkick/query.rb
test/routing_test.rb
test/test_helper.rb
@@ -2,6 +2,7 @@
 - Much better performance for where queries if no facets
 - Added basic support for regex
+- Added support for routing
 ## 0.8.7
@@ -647,6 +647,22 @@ Also supports [additional options](http://www.elasticsearch.org/guide/en/elastic
 City.search "san", boost_by_distance: {field: :location, origin: [37, -122], function: :linear, scale: "30mi", decay: 0.5}
 ```
+### Routing [master]
+
+Searchkick supports [Elasticsearch’s routing feature](https://www.elastic.co/blog/customizing-your-document-routing).
+
+```ruby
+class Contact < ActiveRecord::Base
+  searchkick routing: :user_id
+end
+```
+
+Reindex and search with:
+
+```ruby
+Contact.search "John", routing: current_user.id
+```
+
 ## Inheritance
 Searchkick supports single table inheritance.
@@ -423,9 +423,15 @@ module Searchkick
           }
         end
+        routing = {}
+        if options[:routing]
+          routing = {required: true, path: options[:routing].to_s}
+        end
+
         mappings = {
           _default_: {
             properties: mapping,
+            _routing: routing,
             # https://gist.github.com/kimchy/2898285
             dynamic_templates: [
               {
@@ -379,6 +379,11 @@ module Searchkick
         if options[:type] || klass != searchkick_klass
           @type = [options[:type] || klass].flatten.map { |v| searchkick_index.klass_document_type(v) }
         end
+
+        # routing
+        if options[:routing]
+          @routing = options[:routing]
+        end
       end
       @body = payload
@@ -407,6 +412,7 @@ module Searchkick
         body: body
       }
       params.merge!(type: @type) if @type
+      params.merge!(routing: @routing) if @routing
       params
     end
@@ -0,0 +1,14 @@
+require_relative "test_helper"
+
+class TestRouting < Minitest::Test
+
+  def test_routing_query
+    query = Store.search("Dollar Tree", routing: "Dollar Tree", execute: false)
+    assert_equal query.params[:routing], "Dollar Tree"
+  end
+
+  def test_routing_mappings
+    index_options = Store.searchkick_index.index_options
+    assert_equal index_options[:mappings][:_default_][:_routing], {required: true, path: "name"}
+  end
+end
@@ -212,12 +212,15 @@ class Product
 end
 class Store
-  searchkick mappings: {
-    store: {
-      properties: {
-        name: {type: "string", analyzer: "keyword"}
+  searchkick \
+    routing: :name,
+    merge_mappings: true,
+    mappings: {
+      store: {
+        properties: {
+          name: {type: "string", analyzer: "keyword"},
+        }
       }
-    }
   }
 end
	@@ -2,6 +2,7 @@		@@ -2,6 +2,7 @@
2		2
3	- Much better performance for where queries if no facets	3	- Much better performance for where queries if no facets
4	- Added basic support for regex	4	- Added basic support for regex
		5	+- Added support for routing
5		6
6	## 0.8.7	7	## 0.8.7
7		8
	@@ -647,6 +647,22 @@ Also supports [additional options](http://www.elasticsearch.org/guide/en/elastic		@@ -647,6 +647,22 @@ Also supports [additional options](http://www.elasticsearch.org/guide/en/elastic
647	City.search "san", boost_by_distance: {field: :location, origin: [37, -122], function: :linear, scale: "30mi", decay: 0.5}	647	City.search "san", boost_by_distance: {field: :location, origin: [37, -122], function: :linear, scale: "30mi", decay: 0.5}
648	```	648	```
649		649
		650	+### Routing [master]
		651	+
		652	+Searchkick supports [Elasticsearch’s routing feature](https://www.elastic.co/blog/customizing-your-document-routing).
		653	+
		654	+```ruby
		655	+class Contact < ActiveRecord::Base
		656	+ searchkick routing: :user_id
		657	+end
		658	+```
		659	+
		660	+Reindex and search with:
		661	+
		662	+```ruby
		663	+Contact.search "John", routing: current_user.id
		664	+```
		665	+
650	## Inheritance	666	## Inheritance
651		667
652	Searchkick supports single table inheritance.	668	Searchkick supports single table inheritance.
@@ -0,0 +1,14 @@		@@ -0,0 +1,14 @@
	1	+require_relative "test_helper"
	2	+
	3	+class TestRouting < Minitest::Test
	4	+
	5	+ def test_routing_query
	6	+ query = Store.search("Dollar Tree", routing: "Dollar Tree", execute: false)
	7	+ assert_equal query.params[:routing], "Dollar Tree"
	8	+ end
	9	+
	10	+ def test_routing_mappings
	11	+ index_options = Store.searchkick_index.index_options
	12	+ assert_equal index_options[:mappings][:_default_][:_routing], {required: true, path: "name"}
	13	+ end
	14	+end