elastomer-client 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d77143e192594473f269fc69f9b6552dd7c5b3d8
-  data.tar.gz: aa71ccdf91b2a0277b61595bcd2b34409c3bd02b
+  metadata.gz: ffa99916d4477ed77f1e67b038ddc8de6c70dcf4
+  data.tar.gz: 5700c42af44a02684b86d29d9381cdcacd0aaf50
 SHA512:
-  metadata.gz: b7cb2962c8cdc9712989d9c2e127147e1126ca4f4dcca825c55b4a8f496626f01b236fd59097cb119f031849858f7f1acbe8eda4258e3f9dfa7d7e924533c011
-  data.tar.gz: d780635a018e9d178d28e279ee1051be21ecd522f6db0edcdb88ef53fbef49dc2b2de58433b9b96a1bb13fb773177830487a0f7bbc3e2cd439eb511af6e56268
+  metadata.gz: 6f73679c1b09cafca5499fb515795025528b194cc0794aac1ecdcac2d3a084fa69c7f08f89837de61ecaea1d75d48f9e668d5df15351b8eaa9c8c7246a412f29
+  data.tar.gz: 169c324aac607fd8e3531a2c07aa8191df935390e603d8f7498fec3b093cc2a7a039df6e5280d9bf92018a2dc0e7723a97f02408bb9c90aad3abc88ee90989f9

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 2.0.1 (2016-09-01)
+- Fix bug in delete by query when routing is required
+
+## 2.0.0 (2016-08-25)
+- Support Elasticsearch 2
+
 ## 0.9.0 (2016-02-26)
 - Adding support for the `/_suggest` API endpoint
 - Documentation cleanup - thank you Matt Wagner @wags

data/lib/elastomer/client/delete_by_query.rb

@@ -106,7 +106,7 @@ module Elastomer
       def execute
         ops = Enumerator.new do |yielder|
           @client.scan(@query, @params.merge(:_source => false)).each_document do |hit|
-            yielder.yield([:delete, { :_id => hit["_id"], :_type => hit["_type"], :_index => hit["_index"] }])
+            yielder.yield([:delete, hit.select { |key, _| ["_index", "_type", "_id", "_routing"].include?(key) }])
           end
         end
 

data/lib/elastomer/version.rb

@@ -1,5 +1,5 @@
 module Elastomer
-  VERSION = "2.0.0"
+  VERSION = "2.0.1"
 
   def self.version
     VERSION

data/test/client/delete_by_query_test.rb

@@ -147,5 +147,46 @@ describe Elastomer::Client::DeleteByQuery do
         },
       }, response["_indices"])
     end
+
+    it "deletes by query when routing is specified" do
+      index = $client.index "elastomer-delete-by-query-routing-test"
+      index.delete if index.exists?
+      type = "docs"
+      index.create({ :mappings => { type => { :_routing => { :required => true } } } })
+      wait_for_index(@index.name)
+      docs = index.docs(type)
+
+      docs.index({ :_id => 0, :_routing => "cat", :name => "mittens" })
+      docs.index({ :_id => 1, :_routing => "cat", :name => "luna" })
+
+      index.refresh
+      response = index.delete_by_query(nil, :q => "name:mittens")
+      assert_equal({
+        "_all" => {
+          "found" => 1,
+          "deleted" => 1,
+          "missing" => 0,
+          "failed" => 0,
+        },
+        index.name => {
+          "found" => 1,
+          "deleted" => 1,
+          "missing" => 0,
+          "failed" => 0,
+        },
+      }, response["_indices"])
+
+      index.refresh
+      response = docs.multi_get({
+        :docs => [
+          { :_id => 0, :_routing => "cat" },
+          { :_id => 1, :_routing => "cat" },
+        ]
+      })
+      refute_found response["docs"][0]
+      assert_found response["docs"][1]
+
+      index.delete if index.exists?
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: elastomer-client
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Tim Pease
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-26 00:00:00.000000000 Z
+date: 2016-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -267,7 +267,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.3
+rubygems_version: 2.2.5
 signing_key: 
 specification_version: 4
 summary: A library for interacting with Elasticsearch