poorbarcode commented on code in PR #22705:
URL: https://github.com/apache/pulsar/pull/22705#discussion_r1599398918


##########
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java:
##########
@@ -3493,6 +3495,10 @@ private CompletableFuture<Boolean> 
isAllowAutoTopicCreationAsync(final TopicName
     }
 
     public boolean isDefaultTopicTypePartitioned(final TopicName topicName, 
final Optional<Policies> policies) {
+        if 
(topicName.getPartitionedTopicName().endsWith(DLQ_GROUP_TOPIC_SUFFIX)

Review Comment:
   > Yes, it's a case. What if the user uses the topic name with the -DLQ 
suffix as a regular topic?
   
   - They can use the topics that were created before as before.
   - For new topics, they should create manually if they want a partitioned 
topic with suffix `-DLQ`.



-- 
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-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to