codelipenghui commented on a change in pull request #8564:
URL: https://github.com/apache/pulsar/pull/8564#discussion_r526556808
##########
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/PersistentTopics.java
##########
@@ -1455,6 +1455,7 @@ public void getDeduplicationEnabled(@Suspended final
AsyncResponse asyncResponse
@PathParam("namespace") String namespace,
@PathParam("topic") @Encoded String encodedTopic)
{
validateTopicName(tenant, namespace, encodedTopic);
+ validateTopicOwnership(topicName, true);
Review comment:
we don't need to validate the topic owner ship for topic level policy
since after the policy updated, the broker will get a policy change event, if
the broker own the topic, the broker will apply the policy change to that topic
----------------------------------------------------------------
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]
