[GitHub] [pulsar] ConcurrencyPractitioner commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

[GitBox <gi...@apache.org>]

ConcurrencyPractitioner commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl
URL: https://github.com/apache/pulsar/pull/3983#discussion_r273272433
 
 

 ##########
 File path: pulsar-broker/src/test/java/org/apache/pulsar/client/api/TopicReaderTest.java
 ##########
 @@ -86,6 +87,33 @@ public void testSimpleReader() throws Exception {
         producer.close();
     }
 
+    @Test
+    public void testReaderAtSelectedTimestamp() throws Exception {
+         Producer<byte[]> producer = pulsarClient.newProducer().topic("persistent://my-property/my-ns/testReaderAtSelectedTimestamp")
+                .create();
+         for (int i = 0; i < 10; i++) {
+            String message = "my-message-" + i;
+            producer.send(message.getBytes());
+            Thread.sleep(1000);
 
 Review comment:
   My idea was to use the sleep so that we could space the publish times of the messages 1000ms apart (although that might or might have not worked). 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services