[GitHub] [pulsar] gaoran10 commented on issue #18866: pulsar-perf: improve error message for unrecognized options

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

gaoran10 commented on issue #18866:
URL: https://github.com/apache/pulsar/issues/18866#issuecomment-1345785277

   Maybe the problem is the topics parameter doesn't need a name such as `-t`, `--topics`, so the application can't distinguish whether it's an optional param or not.
   
   For example, if we use this command, the result is the tool will publish messages to the topic `public/default/--foo`.
   ```
   bin/pulsar-perf produce --foo
   ```
   
   If we use this command, the tool wants to publish messages to topics `public/default/--foo` and `public/default/mytopic`, so it prints the tip, `The size of topics list should be equal to --num-topic`.
   ```
   bin/pulsar-perf produce --foo mytopic
   ```
   
   I think if we want to resolve this problem, we need to require the topics parameter to provide a param name, but this will change the using habit.


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