momo-jun commented on code in PR #394:
URL: https://github.com/apache/pulsar-site/pull/394#discussion_r1095306654
##########
docs/reference-pulsar-admin.md:
##########
@@ -1240,26 +1240,29 @@ pulsar-admin namespaces set-backlog-quota
tenant/namespace options
Options
-|Flag|Description|Default|
-|----|---|---|
-|`-l`, `--limit`|The backlog size limit (for example `10M` or `16G`)||
-|`-lt`, `--limitTime`|Time limit in second (or minutes, hours, days, weeks eg:
100m, 3h, 2d, 5w), non-positive number for disabling time limit. (for example
3600 for 1 hour)||
-|`-p`, `--policy`|The retention policy to enforce when the limit is reached.
The valid options are: `producer_request_hold`, `producer_exception` or
`consumer_backlog_eviction`|
-|`-t`, `--type`|Backlog quota type to set. The valid options are:
`destination_storage`, `message_age` |destination_storage|
+| Flag | Description
| Default
|
+|----------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|---------------------|
+| `-t`, `--type` | Backlog quota type to set. The valid options are:
`destination_storage`, `message_age`
|
destination_storage |
Review Comment:
```suggestion
| `-t`, `--type` | The backlog quota type. The valid options are:
`destination_storage` and `message_age`.
|
destination_storage |
```
##########
docs/reference-pulsar-admin.md:
##########
@@ -1240,26 +1240,29 @@ pulsar-admin namespaces set-backlog-quota
tenant/namespace options
Options
-|Flag|Description|Default|
-|----|---|---|
-|`-l`, `--limit`|The backlog size limit (for example `10M` or `16G`)||
-|`-lt`, `--limitTime`|Time limit in second (or minutes, hours, days, weeks eg:
100m, 3h, 2d, 5w), non-positive number for disabling time limit. (for example
3600 for 1 hour)||
-|`-p`, `--policy`|The retention policy to enforce when the limit is reached.
The valid options are: `producer_request_hold`, `producer_exception` or
`consumer_backlog_eviction`|
-|`-t`, `--type`|Backlog quota type to set. The valid options are:
`destination_storage`, `message_age` |destination_storage|
+| Flag | Description
| Default
|
+|----------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|---------------------|
+| `-t`, `--type` | Backlog quota type to set. The valid options are:
`destination_storage`, `message_age`
|
destination_storage |
+| `-l`, `--limit` | If you set backlog quota type to
`destination_storage`, this parameter can set the backlog size limit (for
example `10M` or `16G`), non-positive number for disabling size limit.
||
Review Comment:
```suggestion
| `-l`, `--limit` | If the backlog quota type is set to
`destination_storage`, you can use this parameter to set the backlog size limit
(for example, `10M` or `16G`). To disable the size limit, use a non-positive
number. ||
```
--
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]
