BewareMyPower opened a new pull request, #214: URL: https://github.com/apache/pulsar-client-cpp/pull/214
Fixes https://github.com/apache/pulsar-client-cpp/issues/213 ### Motivation In `ConsumerTest.testAcknowledgeCumulativeWithPartition`, the last two `MessageId`s are acknowledged because they are treated as the latest two `MessageId`s of the two partitions. However, this assumption is false because `MultiTopicsConsumerImpl` does not guarantee receiving messages in a round robin way. Here is an example output when I added the debug logs for the `MessageId` received. ``` (519,48,1,-1) (519,49,1,-1) (518,48,0,-1) (518,49,0,-1) ``` We can see the last two `MessageId`s are both from partition 0. ### Modifications Use an array `latestMsgIds` to record the latest `MessageId` of a given partition in this test. -- 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]
