rdhabalia commented on code in PR #23347:
URL: https://github.com/apache/pulsar/pull/23347#discussion_r1775990470
##########
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:
long option is mostly useful for documentation but the short option is
useful when any user would like to use such tool on-demand and it can be used
without dealing with typos. also, for consistency with other params that have a
combination of short and long options, maybe it's better to keep both options
for this new param as well.
--
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]
