You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/11/17 07:42:25 UTC
[GitHub] [pulsar] labuladong opened a new issue, #18514: [Bug] `pulsar-admin namespaces set-retention` not work
labuladong opened a new issue, #18514:
URL: https://github.com/apache/pulsar/issues/18514
### Search before asking
- [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
### Version
Master version of Pulsar.
### Minimal reproduce step
1. start stanalone Pulsar.
2. use `pulsar-admin namespace set-retention` then `pulsar-admin namespace get-retention`:
### What did you expect to see?
Retention setting has changed.
### What did you see instead?
Retention setting didn't change.
### Anything else?
_No response_
### Are you willing to submit a PR?
- [X] I'm willing to submit a PR!
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] labuladong closed issue #18514: [Bug] `pulsar-admin namespaces set-retention` not work
Posted by GitBox <gi...@apache.org>.
labuladong closed issue #18514: [Bug] `pulsar-admin namespaces set-retention` not work
URL: https://github.com/apache/pulsar/issues/18514
--
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
[GitHub] [pulsar] labuladong commented on issue #18514: [Bug] `pulsar-admin namespaces set-retention` not work
Posted by GitBox <gi...@apache.org>.
labuladong commented on issue #18514:
URL: https://github.com/apache/pulsar/issues/18514#issuecomment-1319431078
This is not a cli bug, because the `--size` and `--time` need to specify a unit. The default is `byte` and `second`, so it will be considered as 0 if too small:
```shell
Usage: set-retention [options] tenant/namespace
Options:
* --size, -s
Retention size limit with optional size unit suffix. For example, 4096,
10M, 16G, 3T. The size unit suffix character can be k/K, m/M, g/G, or
t/T. If the size unit suffix is not specified, the default unit is
bytes. 0 or less than 1MB means no retention and -1 means infinite size
retention
* --time, -t
Retention time with optional time unit suffix. For example, 100m, 3h,
2d, 5w. If the time unit is not specified, the default unit is seconds.
For example, -t 120 sets retention to 2 minutes. 0 means no retention
and -1 means infinite time retention.
```
But the site document is wrong, so fix doc in #18526.
--
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
[GitHub] [pulsar] tisonkun closed issue #18514: [Bug] `pulsar-admin namespaces set-retention` not work
Posted by GitBox <gi...@apache.org>.
tisonkun closed issue #18514: [Bug] `pulsar-admin namespaces set-retention` not work
URL: https://github.com/apache/pulsar/issues/18514
--
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