315157973 commented on a change in pull request #9530:
URL: https://github.com/apache/pulsar/pull/9530#discussion_r573380210



##########
File path: site2/docs/reference-pulsar-admin.md
##########
@@ -2267,35 +2230,28 @@ Options
 |`-l`, `--ledgerId`|The ledger id |0|
 |`-e`, `--entryId`|The entry id |0|
 
-### `get-backlog-quotas`
-Get the backlog quota policies for a topic.
+### `enable-deduplication`
+Enable a deduplication policy for a topic.
 
 Usage
 ```bash
-$ pulsar-admin topics get-backlog-quotas tenant/namespace/topic
+$ pulsar-admin topics enable-deduplication tenant/namespace/topic
 ```
 
-### `set-backlog-quota`
-Set a backlog quota policy for a topic.
-
-Usage
-```bash
-$ pulsar-admin topics set-backlog-quota tenant/namespace/topic options
-=======
-### `get-deduplication`
-Get a deduplication policy for a topic.
+### `disable-deduplication`

Review comment:
       Based the code on the master branch, in the issue mentioned by 
@Jennifer88huang , the command names of topic-level and namespace-level have 
been unified, https://github.com/apache/pulsar/issues/9205
   
   E.g:
   The topic level `get-maxProducers` has been changed to `get-max-producers`, 
and `get-maxProducers` is marked as deprecated.
   
   The deprecated command will not be deleted immediately, it can still be 
used, but it will no longer be displayed when user enters `--help`, and may be 
deleted at some point in the future.
   
   Since the code is in the master branch, one thing that needs to be confirmed 
is whether this PR will be cherry-picked to 2.7.x and the document should 
correspond to it  @codelipenghui 




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to