[improve][broker] Make Consumer#equals more effective (apache#18662)

(cherry picked from commit 9e821b0) (cherry picked from commit d2b3986)
datastax · Jan 10, 2023 · 38adf57 · 38adf57
1 parent 72857e4
commit 38adf57
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/Consumer.java b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/Consumer.java
@@ -876,7 +876,7 @@ public CompletableFuture<Void> checkPermissionsAsync() {
     public boolean equals(Object obj) {
         if (obj instanceof Consumer) {
             Consumer other = (Consumer) obj;
-            return Objects.equals(cnx.clientAddress(), other.cnx.clientAddress()) && consumerId == other.consumerId;
+            return consumerId == other.consumerId && Objects.equals(cnx.clientAddress(), other.cnx.clientAddress());
         }
         return false;
     }