Re: [PR] feat: introduce ack trackers to prevent batch message loss [pulsar-client-go]

2024-09-06 Thread via GitHub
KennyChenFight commented on PR #1277: URL: https://github.com/apache/pulsar-client-go/pull/1277#issuecomment-2335051805 > > the messages added to ack_grouping_tracker are considered ready to be acknowledged. > > You can create a map to record the batch message, such as `batchMessage

Re: [PR] feat: introduce ack trackers to prevent batch message loss [pulsar-client-go]

2024-09-04 Thread via GitHub
nodece commented on PR #1277: URL: https://github.com/apache/pulsar-client-go/pull/1277#issuecomment-2330580987 Good work! I think we should fix this issue in the `ack_grouping_tracker.go`. When acknowledging a batch message, ensure that the ledger ID, entry ID, and tracker a