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

xyz 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 ee202d0  [websocket] Remove System.out.println from ConsumerHandler 
(#11459)
ee202d0 is described below

commit ee202d06548e3c73d70ad52374658ee3507ca809
Author: Masahiro Sakamoto <[email protected]>
AuthorDate: Tue Jul 27 15:26:10 2021 +0900

    [websocket] Remove System.out.println from ConsumerHandler (#11459)
    
    `System.out.println` is called in the code of WebSocket proxy. This seems 
to be for debugging and I think it should be removed.
---
 .../src/main/java/org/apache/pulsar/websocket/ConsumerHandler.java       | 1 -
 1 file changed, 1 deletion(-)

diff --git 
a/pulsar-websocket/src/main/java/org/apache/pulsar/websocket/ConsumerHandler.java
 
b/pulsar-websocket/src/main/java/org/apache/pulsar/websocket/ConsumerHandler.java
index c798739..525ea0f 100644
--- 
a/pulsar-websocket/src/main/java/org/apache/pulsar/websocket/ConsumerHandler.java
+++ 
b/pulsar-websocket/src/main/java/org/apache/pulsar/websocket/ConsumerHandler.java
@@ -283,7 +283,6 @@ public class ConsumerHandler extends 
AbstractWebSocketHandler {
     private void handleNack(ConsumerCommand command) throws IOException {
         MessageId msgId = 
MessageId.fromByteArrayWithTopic(Base64.getDecoder().decode(command.messageId),
             topic.toString());
-        System.out.println(msgId);
         consumer.negativeAcknowledge(msgId);
         checkResumeReceive();
     }

Reply via email to