lhotari commented on code in PR #23347:
URL: https://github.com/apache/pulsar/pull/23347#discussion_r1774385245


##########
pulsar-client-tools/src/main/java/org/apache/pulsar/client/cli/CmdRead.java:
##########
@@ -101,6 +102,9 @@ public class CmdRead extends AbstractCmdConsume {
     @Option(names = { "-pm", "--pool-messages" }, description = "Use the 
pooled message", arity = "1")
     private boolean poolMessages = true;
 
+    @Option(names = { "-mp", "--print-metadata" }, description = "Message 
metadata")

Review Comment:
   `-mp` isn't very intuitive. perhaps there doesn't need to be a short option 
and `--print-metadata` would be sufficient? (no strong opinion on this)



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to