[GitHub] [pulsar] Anonymitaet commented on a change in pull request #12016: [testclient] Add new option --num-messages for consumer and reader

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

Anonymitaet commented on a change in pull request #12016:
URL: https://github.com/apache/pulsar/pull/12016#discussion_r707146294



##########
File path: site2/docs/reference-cli-tools.md
##########
@@ -437,6 +437,7 @@ Options
 |`-v`, `--encryption-key-value-file`|The file which contains the private key to decrypt payload||
 |`-h`, `--help`|Help message|false|
 |`--conf-file`|Configuration file||
+|`-m`, `--num-messages`|Number of messages to consume in total. If <= 0, it will keep consuming.|0|

Review comment:
       Please keep consistent in other descriptions.

##########
File path: site2/docs/reference-cli-tools.md
##########
@@ -437,6 +437,7 @@ Options
 |`-v`, `--encryption-key-value-file`|The file which contains the private key to decrypt payload||
 |`-h`, `--help`|Help message|false|
 |`--conf-file`|Configuration file||
+|`-m`, `--num-messages`|Number of messages to consume in total. If <= 0, it will keep consuming.|0|

Review comment:
       ```suggestion
   |`-m`, `--num-messages`|Number of messages to consume in total. If the value is equal to or smaller than 0, it keeps consuming messages.|0|
   ```




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