From c72960e30abed9a82a8688ab46f3546d620e998c Mon Sep 17 00:00:00 2001
From: Robert Newson <rnewson@apache.org>
Date: Mon, 25 Mar 2024 09:12:43 +0000
Subject: [PATCH] nouveau: fix rexi_DOWN clause

reported in https://github.com/apache/couchdb/issues/5002

the badrecord is because we execute the second clause of handle_message
and mess up the internal state (just returning the Counters rather than a
state record around it)
---
 src/nouveau/src/nouveau_fabric_search.erl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/nouveau/src/nouveau_fabric_search.erl b/src/nouveau/src/nouveau_fabric_search.erl
index d11b55670d8..5008df9d6e9 100644
--- a/src/nouveau/src/nouveau_fabric_search.erl
+++ b/src/nouveau/src/nouveau_fabric_search.erl
@@ -101,7 +101,7 @@ handle_message({rexi_DOWN, _, {_, NodeRef}, _}, _Shard, State) ->
     #state{counters = Counters0} = State,
     case fabric_util:remove_down_workers(Counters0, NodeRef, []) of
         {ok, Counters1} ->
-            {ok, Counters1};
+            {ok, State#state{counters = Counters1}};
         error ->
             {error, {nodedown, <<"progress not possible">>}}
     end;