michaeljmarshall commented on code in PR #20245:
URL: https://github.com/apache/pulsar/pull/20245#discussion_r1218261086
##########
pulsar-broker/src/test/java/org/apache/pulsar/client/api/SimpleProducerConsumerTest.java:
##########
@@ -4109,6 +4109,35 @@ public void testGetStats() throws Exception {
consumer.close();
producer.close();
}
+ @Test(timeOut = 100000)
+ public void testMessageListenerGetStats() throws Exception {
+ final String topicName = "persistent://my-property/my-ns/testGetStats"
+ UUID.randomUUID();
+ final String subName = "my-sub";
+ final int receiveQueueSize = 100;
+ @Cleanup
+ PulsarClient client = newPulsarClient(lookupUrl.toString(), 100);
+ Producer<String> producer = pulsarClient.newProducer(Schema.STRING)
+ .enableBatching(false).topic(topicName).create();
+ ConsumerImpl<String> consumer = (ConsumerImpl<String>)
client.newConsumer(Schema.STRING)
+ .messageListener((MessageListener<String>) (consumer1, msg) ->
{
+ try {
+ TimeUnit.SECONDS.sleep(10);
Review Comment:
Pulsar tests take a long time to run, so it would be helpful to try to find
faster ways to test features to prevent them taking even longer. This test is
only verifying correct counting of a queue. Perhaps a faster solution might use
futures that are completed at specific locations in the test to verify the
behavior.
--
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]
