Technoboy- commented on a change in pull request #13997:
URL: https://github.com/apache/pulsar/pull/13997#discussion_r794270617



##########
File path: 
pulsar-broker/src/test/java/org/apache/pulsar/client/api/BrokerServiceLookupTest.java
##########
@@ -402,8 +402,6 @@ public void testPartitionTopicLookup() throws Exception {
         producer.close();
         consumer.unsubscribe();
         consumer.close();
-        admin.topics().deletePartitionedTopic(topicName.toString());

Review comment:
       Could we use `deletePartitionedTopicAsync`?
   For if we delete this line and the test execute with retry, could it occur 
the 'the topic existed' exception?




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


Reply via email to