[GitHub] [pulsar-client-go] merlimat opened a new pull request #90: Removed unacked messages tracker

[GitBox <gi...@apache.org>]

merlimat opened a new pull request #90: Removed unacked messages tracker
URL: https://github.com/apache/pulsar-client-go/pull/90
 
 
   ### Motivation
   
   Ack timeout is legacy feature that is offered in Java and C++ client libs, though it's not the best way to solve the problem. (backstory: this came out basically by implementing what the user was saying rather than filtering that and solving the bigger root cause).
   
   Negative acks are better at that because apps can directly signal the success/failure in processing a particular message. Ack timeout cannot be turned on by default since we don't know what's the max legit processing time for a message.
   
   For that, since we're starting a client lib with no legacy to support, we should avoid exposing ack timeouts all-together.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services