[incubator-pulsar] branch master updated: Documentation need to be updated for Admin command to apply deduplication at namespace: (#2157)

[sijie]

This is an automated email from the ASF dual-hosted git repository.

sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new 300f736  Documentation need to be updated for Admin command to apply 
deduplication at namespace: (#2157)
300f736 is described below

commit 300f736fced35f13f931f70ca90e7f1b65e75e34
Author: Karthikeyan Palanivelu <pcke...@users.noreply.github.com>
AuthorDate: Sat Jul 14 14:02:45 2018 -0400

    Documentation need to be updated for Admin command to apply deduplication 
at namespace: (#2157)
    
    bin/pulsar-admin namespaces set-deduplication \
      persistent://public/default/topic-1 \
      --enable # or just -e
    
    should be
    
    bin/pulsar-admin namespaces set-deduplication \
     public/default \
      --enable # or just -e
    
    How can we reproduce the issue
    
    Running the below statement will result in
    
    java.lang.IllegalArgumentException: Invalid namespace format. namespace: 
persistent://public/default/topic-1
    
    **Pulsar version**: 2.0-rc1
    
    Fixes: #2156
---
 site/docs/latest/cookbooks/message-deduplication.md | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/site/docs/latest/cookbooks/message-deduplication.md 
b/site/docs/latest/cookbooks/message-deduplication.md
index 9a63264..ea6fa64 100644
--- a/site/docs/latest/cookbooks/message-deduplication.md
+++ b/site/docs/latest/cookbooks/message-deduplication.md
@@ -36,21 +36,21 @@ Regardless of the value of `brokerDeduplicationEnabled`, 
[enabling](#enabling) a
 
 ### Enabling message deduplication {#enabling}
 
-You can enable message deduplication on specific namespaces, regardless of the 
the [default](#default) for the broker, using the [`pulsar-admin namespace 
set-deduplication`](../../CliTools#pulsar-admin-namespace-set-deduplication) 
command. You can use the `--enable`/`-e` flag and specify the namespace. Here's 
an example:
+You can enable message deduplication on specific namespaces, regardless of the 
the [default](#default) for the broker, using the [`pulsar-admin namespace 
set-deduplication`](../../CliTools#pulsar-admin-namespace-set-deduplication) 
command. You can use the `--enable`/`-e` flag and specify the namespace. Here's 
an example with <tenant>/<namespace>:
 
 ```bash
 $ bin/pulsar-admin namespaces set-deduplication \
-  persistent://public/default/topic-1 \
+  public/default \
   --enable # or just -e
 ```
 
 ### Disabling message deduplication {#disabling}
 
-You can disable message deduplication on a specific namespace using the same 
method shown [above](#enabling), except using the `--disable`/`-d` flag 
instead. Here's an example:
+You can disable message deduplication on a specific namespace using the same 
method shown [above](#enabling), except using the `--disable`/`-d` flag 
instead. Here's an example with <tenant>/<namespace>:
 
 ```bash
 $ bin/pulsar-admin namespaces set-deduplication \
-  persistent://public/default/topic-1 \
+  public/default \
   --disable # or just -d
 ```