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

technoboy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new 2dace760b03 [improve][client] Add log when can't add message to the 
container (#21924)
2dace760b03 is described below

commit 2dace760b03bd679dae5ea682e1e23fd93c15f78
Author: Jiwei Guo <[email protected]>
AuthorDate: Fri Oct 11 09:16:50 2024 +0800

    [improve][client] Add log when can't add message to the container (#21924)
---
 .../src/main/java/org/apache/pulsar/client/impl/MessagesImpl.java   | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessagesImpl.java 
b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessagesImpl.java
index d4cd36a22e1..9768fd7c74b 100644
--- 
a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessagesImpl.java
+++ 
b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessagesImpl.java
@@ -22,11 +22,13 @@ import static 
com.google.common.base.Preconditions.checkArgument;
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
+import lombok.extern.slf4j.Slf4j;
 import net.jcip.annotations.NotThreadSafe;
 import org.apache.pulsar.client.api.Message;
 import org.apache.pulsar.client.api.Messages;
 
 @NotThreadSafe
+@Slf4j
 public class MessagesImpl<T> implements Messages<T> {
 
     private final List<Message<T>> messageList;
@@ -49,10 +51,14 @@ public class MessagesImpl<T> implements Messages<T> {
             return true;
         }
         if (maxNumberOfMessages > 0 && currentNumberOfMessages + 1 > 
maxNumberOfMessages) {
+            log.warn("can't add message to the container, has exceeded the 
maxNumberOfMessages : {} ",
+                    maxNumberOfMessages);
             return false;
         }
 
         if (maxSizeOfMessages > 0 && currentSizeOfMessages + message.size() > 
maxSizeOfMessages) {
+            log.warn("can't add message to the container, has exceeded the 
maxSizeOfMessages : {} ",
+                    maxSizeOfMessages);
             return false;
         }
 

Reply via email to