poorbarcode opened a new pull request, #20568:
URL: https://github.com/apache/pulsar/pull/20568

   ### Motivation
   If sent messages to the client fail by the IO exception, these messages will 
be redelivered after the heartbeat check fails(it triggers connection close). 
And if the next heartbeat check is successful, these messages will be stuck 
until the client reconnects or the topic is reloaded. see: 
https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/Consumer.java#L364
   
   ### Modifications
   - print log if sending messages to client failed by IO EX.
   - add `consumer-id` into the log when doing subscribe(`cnx.address + 
consumerId` is the identifier of one consumer)
   
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes. -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update 
later -->
   - [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   
   ### Matching PR in forked repository
   
   PR in forked repository: x
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to