From baa01d4d9911981e81f66b89234c6928a234b268 Mon Sep 17 00:00:00 2001 From: Harshitha Sudhakar <111514477+harshithasudhakar@users.noreply.github.com> Date: Wed, 5 Apr 2023 17:04:18 +0530 Subject: [PATCH] Update ClientSessionGroupMapping.java Changed "size()>0" to "isEmpty()" --- .../protocol/tcp/client/group/ClientSessionGroupMapping.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/tcp/client/group/ClientSessionGroupMapping.java b/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/tcp/client/group/ClientSessionGroupMapping.java index 66b62f3a61..a0c0880088 100644 --- a/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/tcp/client/group/ClientSessionGroupMapping.java +++ b/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/tcp/client/group/ClientSessionGroupMapping.java @@ -294,7 +294,7 @@ private void cleanSubscriptionInSession(Session session) throws Exception { private void handleUnackMsgsInSession(Session session) { ConcurrentHashMap unAckMsg = session.getPusher().getUnAckMsg(); ClientGroupWrapper clientGroupWrapper = Objects.requireNonNull(session.getClientGroupWrapper().get()); - if (unAckMsg.size() > 0 && clientGroupWrapper.getGroupConsumerSessions().size() > 0) { + if (unAckMsg.size() > 0 && !clientGroupWrapper.getGroupConsumerSessions().isEmpty()) { for (Map.Entry entry : unAckMsg.entrySet()) { DownStreamMsgContext downStreamMsgContext = entry.getValue(); if (SubscriptionMode.BROADCASTING == downStreamMsgContext.getSubscriptionItem().getMode()) {