lhotari commented on issue #7796:
URL: https://github.com/apache/pulsar/issues/7796#issuecomment-692014328


   > I have made some modify and pushed again.
   
   @aloyszhang Looks promising. Good work. 
   
   Would it be possible to add test cases with ordinary message id seeks? For 
example a test case where there would be a fixed number of messages in a topic 
(for example just 3 message). When publishing the messages to the topic, their 
respective MessageIds could be stored. The test would then do a given number of 
reads (for example 100 times) where it first uses seek to go to one of the 
randomly picked message ids, checks with hasMessageAvailable that there's a 
message available and then reads the message and verifies that it's the correct 
message. This should be enough to show that the problem with seek + 
hasMessageAvailable has been resolved and doesn't run into some race conditions 
caused by the broker code that "disconnects all the connected consumers and 
then resets the cursor" (as described in #5278).
   


----------------------------------------------------------------
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]


Reply via email to