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

jinrongtong pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/rocketmq.git


The following commit(s) were added to refs/heads/develop by this push:
     new 2e8ef0464 [ISSUE #6043] Optimize the DefaultLitePullConsumerImpl 
isSetEqual method code (#6044)
2e8ef0464 is described below

commit 2e8ef046465c4133cf0d6ad6f242f630021439b2
Author: hardyfish <[email protected]>
AuthorDate: Mon Feb 13 13:38:30 2023 +0800

    [ISSUE #6043] Optimize the DefaultLitePullConsumerImpl isSetEqual method 
code (#6044)
    
    * optimize the DefaultLitePullConsumerImpl isSetEqual method code
    
    * optimize the DefaultLitePullConsumerImpl isSetEqual method code
---
 .../rocketmq/client/impl/consumer/DefaultLitePullConsumerImpl.java   | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/client/src/main/java/org/apache/rocketmq/client/impl/consumer/DefaultLitePullConsumerImpl.java
 
b/client/src/main/java/org/apache/rocketmq/client/impl/consumer/DefaultLitePullConsumerImpl.java
index 793778e03..c6631cb5e 100644
--- 
a/client/src/main/java/org/apache/rocketmq/client/impl/consumer/DefaultLitePullConsumerImpl.java
+++ 
b/client/src/main/java/org/apache/rocketmq/client/impl/consumer/DefaultLitePullConsumerImpl.java
@@ -1215,12 +1215,11 @@ public class DefaultLitePullConsumerImpl implements 
MQConsumerInner {
             return true;
         }
 
-        if (set1 == null || set2 == null || set1.size() != set2.size()
-            || set1.size() == 0 || set2.size() == 0) {
+        if (set1 == null || set2 == null || set1.size() != set2.size() || 
set1.size() == 0) {
             return false;
         }
 
-        Iterator iter = set2.iterator();
+        Iterator<MessageQueue> iter = set2.iterator();
         boolean isEqual = true;
         while (iter.hasNext()) {
             if (!set1.contains(iter.next())) {

Reply via email to