Skip to content

Commit 678bf5b

Browse files
authored
Merge pull request #323 from github/fix-params-merges
Removing redundant params in params.merge()
2 parents 653bb93 + 9875340 commit 678bf5b

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

lib/elastomer_client/client/reindex.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,12 @@ def initialize(client)
2020
attr_reader :client
2121

2222
def reindex(body, params = {})
23-
response = client.post "/_reindex", params.merge(params, body:, action: "reindex", rest_api: "reindex")
23+
response = client.post "/_reindex", params.merge(body:, action: "reindex", rest_api: "reindex")
2424
response.body
2525
end
2626

2727
def rethrottle(task_id, params = {})
28-
response = client.post "/_reindex/#{task_id}/_rethrottle", params.merge(params, action: "rethrottle", rest_api: "reindex")
28+
response = client.post "/_reindex/#{task_id}/_rethrottle", params.merge(action: "rethrottle", rest_api: "reindex")
2929
response.body
3030
end
3131

0 commit comments

Comments
 (0)