is_recoverable/1: improve guards

michaelklishin · mergify[bot] · commit 2f312d85e152 · 2023-07-05T08:42:34.000Z
(cherry picked from commit 8dfcc9c)
diff --git a/deps/rabbit/src/rabbit_classic_queue.erl b/deps/rabbit/src/rabbit_classic_queue.erl
@@ -121,7 +121,7 @@ delete(Q, IfUnused, IfEmpty, ActingUser) when ?amqqueue_is_classic(Q) ->
             {ok, 0}
     end.
 
-is_recoverable(Q) when ?is_amqqueue(Q) ->
+is_recoverable(Q) when ?is_amqqueue(Q) and ?amqqueue_is_classic(Q) ->
     Node = node(),
     Node =:= amqqueue:qnode(Q) andalso
     %% Terminations on node down will not remove the rabbit_queue
diff --git a/deps/rabbit/src/rabbit_quorum_queue.erl b/deps/rabbit/src/rabbit_quorum_queue.erl
@@ -554,7 +554,7 @@ reductions(Name) ->
             0
     end.
 
-is_recoverable(Q) ->
+is_recoverable(Q) when ?is_amqqueue(Q) and ?amqqueue_is_quorum(Q) ->
     Node = node(),
     Nodes = get_nodes(Q),
     lists:member(Node, Nodes).

Original file line number	Diff line number	Diff line change
`@@ -554,7 +554,7 @@ reductions(Name) ->`
`554`	`554`	`0`
`555`	`555`	`end.`
`556`	`556`
`557`		`-is_recoverable(Q) ->`
	`557`	`+is_recoverable(Q) when ?is_amqqueue(Q) and ?amqqueue_is_quorum(Q) ->`
`558`	`558`	`Node = node(),`
`559`	`559`	`Nodes = get_nodes(Q),`
`560`	`560`	`lists:member(Node, Nodes).`