MarvinCai commented on a change in pull request #5148: Validate topic name
before creating partition/non partition topic via admin cli.
URL: https://github.com/apache/pulsar/pull/5148#discussion_r334737624
##########
File path:
pulsar-broker/src/test/java/org/apache/pulsar/client/impl/TopicsConsumerImplTest.java
##########
@@ -685,92 +685,6 @@ public void testMultiTopicsMessageListener() throws
Exception {
consumer.close();
}
-
- /**
- * Test topic partitions auto subscribed.
- *
- * Steps:
- * 1. Create a consumer with 2 topics, and each topic has 2 partitions:
xx-partition-0, xx-partition-1.
- * 2. produce message to xx-partition-2, and verify consumer could not
receive message.
Review comment:
sure, updated to keep test case to still test can auto subscribe newly added
partitions.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
