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/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 8aee353 update topicName document to avoid confusion (#6816)
8aee353 is described below
commit 8aee353d5b000c80be5cc281e6b574bd62ba5531
Author: hangc0276 <[email protected]>
AuthorDate: Wed Apr 29 12:27:51 2020 +0800
update topicName document to avoid confusion (#6816)
Fix #6620
### changes
1. update topic name document
2. update code annotation in TopicName.java to avoid topic name confusion
---
.../src/main/java/org/apache/pulsar/common/naming/TopicName.java | 4 ++--
site2/docs/getting-started-pulsar.md | 1 +
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git
a/pulsar-common/src/main/java/org/apache/pulsar/common/naming/TopicName.java
b/pulsar-common/src/main/java/org/apache/pulsar/common/naming/TopicName.java
index 2b5a1e3..5ec0d58 100644
--- a/pulsar-common/src/main/java/org/apache/pulsar/common/naming/TopicName.java
+++ b/pulsar-common/src/main/java/org/apache/pulsar/common/naming/TopicName.java
@@ -136,8 +136,8 @@ public class TopicName implements ServiceUnitId {
// new: tenant/namespace/<localName>
// legacy: tenant/cluster/namespace/<localName>
// Examples of localName:
- // 1. some/name/xyz//
- // 2. /xyz-123/feeder-2
+ // 1. some, name, xyz
+ // 2. xyz-123, feeder-2
parts = Splitter.on("/").limit(4).splitToList(rest);
diff --git a/site2/docs/getting-started-pulsar.md
b/site2/docs/getting-started-pulsar.md
index 043ac4b..6ba815e 100644
--- a/site2/docs/getting-started-pulsar.md
+++ b/site2/docs/getting-started-pulsar.md
@@ -32,6 +32,7 @@ Two important changes have been made in Pulsar 2.0:
* There is no longer a [cluster component](#no-cluster)
* Properties have been [renamed to tenants](#tenants)
* You can use a [flexible](#flexible-topic-naming) naming system to shorten
many topic names
+* `/` is not allowed in topic name
#### No cluster component
