[GitHub] [pulsar] yuruguo commented on a change in pull request #12090: [pulsar-perf] Make it possible to disable poolMessages

[GitBox <gi...@apache.org>]

yuruguo commented on a change in pull request #12090:
URL: https://github.com/apache/pulsar/pull/12090#discussion_r711592306



##########
File path: pulsar-testclient/src/main/java/org/apache/pulsar/testclient/PerformanceConsumer.java
##########
@@ -176,11 +176,11 @@
         @Parameter(names = {"-ioThreads", "--num-io-threads"}, description = "Set the number of threads to be " +
                 "used for handling connections to brokers, default is 1 thread")
         public int ioThreads = 1;
-    
+
         @Parameter(names = {"--batch-index-ack" }, description = "Enable or disable the batch index acknowledgment")
         public boolean batchIndexAck = false;
 
-        @Parameter(names = { "-pm", "--pool-messages" }, description = "Use the pooled message")
+        @Parameter(names = { "-pm", "--pool-messages" }, description = "Use the pooled message", arity = 1)

Review comment:
       Good catch！Should other option that type is boolean be handled like this? such as --replicated, --auto_ack_chunk_q_full, --batch-index-ack etc.




-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org