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 are properly managed within a map. The logic for handling the tracker is as follows: - New batch Message without a Tracker: If the incoming batch message does not have a tracker but a matching ledger ID and entry ID exist in the map, use the tracker already stored in the map. - New batch Message with a Tracker: If the incoming batch message has a tracker but it differs from the one already in the map for the same ledger ID and entry ID, merge the two trackers to ensure consistency. What do you think? -- 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]
