eolivelli commented on a change in pull request #10963:
URL: https://github.com/apache/pulsar/pull/10963#discussion_r655245674



##########
File path: 
pulsar-broker/src/test/java/org/apache/pulsar/broker/BrokerTestUtil.java
##########
@@ -29,4 +31,8 @@ public static String newUniqueName(String prefix) {
         return prefix + "-" + UUID.randomUUID();
     }
 
+    public static String randomSuffixString(String content, int numSuffix) {
+        return content + "-" + 
RandomStringUtils.randomAlphabetic(numSuffix).toLowerCase();

Review comment:
       we can enhance newUniqueName in a follow up patch if you prefer.
   at the moment it is better to have only one single way of generating random 
topic names
   




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