zymap commented on a change in pull request #8535:
URL: https://github.com/apache/pulsar/pull/8535#discussion_r521970529
##########
File path:
pulsar-testclient/src/main/java/org/apache/pulsar/proxy/socket/client/PerformanceClient.java
##########
@@ -166,7 +168,9 @@ public void runPerformanceTest(long messages, long limit,
int numOfTopic, int si
String topicName, String authPluginClassName, String authParams)
throws InterruptedException, FileNotFoundException {
ExecutorService executor = Executors.newCachedThreadPool(new
DefaultThreadFactory("pulsar-perf-producer-exec"));
HashMap<String, Tuple> producersMap = new HashMap<>();
- String produceBaseEndPoint = baseUrl + "ws/producer" + topicName;
+ String replacedTopicName = topicName.replace("://", "/");
Review comment:
I think there has a method `getRestPath` in the TopicName do the same
thing with you.
----------------------------------------------------------------
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]
