nicoloboschi commented on code in PR #18804:
URL: https://github.com/apache/pulsar/pull/18804#discussion_r1044173204


##########
pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApi2Test.java:
##########
@@ -1648,6 +1648,11 @@ public void testDeleteNamespaceWithTopicPolicies() 
throws Exception {
 
         // create namespace2
         String namespace = tenant + "/test-ns2";
+        admin.namespaces().createNamespace(namespace, Set.of("test"));
+        admin.topics().createNonPartitionedTopic(namespace + "/tobedeleted");
+        // verify namespace can be deleted even without topic policy events

Review Comment:
   it's the scope of this pull 😁 
   When deleting namespace with force=true, you first delete the topics in it. 
For each topics deleted, a new event is generated. However if the 
__change_events doesn't exist yet, it's auto-created. But the creation fails 
since the namespace is being deleted.



-- 
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: [email protected]

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

Reply via email to