[couchdb] 01/01: Remove reduntant get_ring_opts call in in `dreyfus_fabric_search`

[va...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

vatamane pushed a commit to branch remove-duplicate-get-ring-ops-in-dreyfus
in repository https://gitbox.apache.org/repos/asf/couchdb.git

commit 5267f34926e2b0eb49f754cac1a93f99776360df
Author: Nick Vatamaniuc <va...@apache.org>
AuthorDate: Thu Jan 23 09:43:43 2020 -0500

    Remove reduntant get_ring_opts call in in `dreyfus_fabric_search`
    
    Previously we erroneously got ring opts before and after figuring out live
    shards if some shards were not live there was a badmatch failure.
---
 src/dreyfus/src/dreyfus_fabric_search.erl | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/dreyfus/src/dreyfus_fabric_search.erl b/src/dreyfus/src/dreyfus_fabric_search.erl
index c43a98d..8edaa38 100644
--- a/src/dreyfus/src/dreyfus_fabric_search.erl
+++ b/src/dreyfus/src/dreyfus_fabric_search.erl
@@ -56,7 +56,6 @@ go(DbName, DDoc, IndexName, #index_query_args{}=QueryArgs) ->
     Shards = dreyfus_util:get_shards(DbName, QueryArgs),
     LiveNodes = [node() | nodes()],
     LiveShards = [S || #shard{node=Node} = S <- Shards, lists:member(Node, LiveNodes)],
-    RingOpts = dreyfus_util:get_ring_opts(QueryArgs, LiveShards),
     Bookmark1 = dreyfus_bookmark:add_missing_shards(Bookmark0, LiveShards),
     Counters0 = lists:flatmap(fun({#shard{name=Name, node=N} = Shard, After}) ->
         QueryArgs1 = dreyfus_util:export(QueryArgs#index_query_args{