wuYin opened a new pull request #371: URL: https://github.com/apache/pulsar-client-go/pull/371
# Motivation Follow doc [acknowledgement-timeout](https://pulsar.apache.org/docs/en/concepts-messaging/#acknowledgement-timeout) to support ack timeout feature. ### Modifications - Add `AckTimeout` to ConsumerOptions. - Add `unackedMessageTracker` to track messages which received by user. - Add 5 `TestAckTimeout*` test cases. ### Verifying this change - [ ] Make sure that the change passes the CI checks. This change added tests and can be verified as follows: - `TestAckTimeout*` Track ack timeout in single topic consumers, with `Exclusive, Failover, Shared` subscription. - `TestAckTimeoutSharedMultiAndRegexTopics` Track ack timeout in multi topics consumers, with `Shared` subscription. ### Different default behavior: - Java Client: if consumer enabled DLQPolicy, added 30s `ackTimeout` by default, see [apache/pulsar #3104](https://github.com/apache/pulsar/pull/3014) - Go Client: I think it's no longer necessary to do this, so ignored. ---------------------------------------------------------------- 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: [email protected]
