shibd commented on issue #14516: URL: https://github.com/apache/pulsar/issues/14516#issuecomment-1058792886
Sorry, I didn't reproduce for local. I looked at the test code. It should be that the following code sequence is wrong. Before delete a namespace, We should mock all done. 847 lines of code should be executed after 851 lines of code. https://github.com/apache/pulsar/blob/4f1e39b6921ea401b8c27f17a041d06d85f8abf8/pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/NamespacesTest.java#L847-L851 Otherwise, a null pointer may happen during verification `isServiceUnitOwned` https://github.com/apache/pulsar/blob/4f1e39b6921ea401b8c27f17a041d06d85f8abf8/pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java#L584 Final throw RestException https://github.com/apache/pulsar/blob/4f1e39b6921ea401b8c27f17a041d06d85f8abf8/pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java#L608-L610 @Jason918 PTAL -- 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]
