rdhabalia commented on a change in pull request #10184:
URL: https://github.com/apache/pulsar/pull/10184#discussion_r616916831



##########
File path: 
pulsar-client-tools/src/main/java/org/apache/pulsar/client/cli/CmdConsume.java
##########
@@ -122,7 +123,9 @@
     @Parameter(names = { "-st", "--schema-type"}, description = "Set a schema 
type on the consumer, it can be 'bytes' or 'auto_consume'")
     private String schematype = "bytes";
 
-    
+    @Parameter(names = { "-pm", "--pool-messages" }, description = "Use the 
pooled message")
+    private boolean poolMessages = false;

Review comment:
       I changed default value to true. let's keep this configuration for a 
release because for a small memory size client-host, they might want to use 
heap-memory as GC can help to make some space when internal queue size tuning 
is not right and all the memory is used.




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