[GitHub] [pulsar] dlg99 commented on pull request #19374: [fix][broker] Fix delete namespace fail by a race condition

["dlg99 (via GitHub)" <gi...@apache.org>]

dlg99 commented on PR #19374:
URL: https://github.com/apache/pulsar/pull/19374#issuecomment-1410904484

   @mattisonchao can this be done slightly differently:
   swap order of steps 2 and 3 (mark deleted first, do the rest, unmark on error).
   In combo with https://github.com/apache/pulsar/pull/19097 this should work good enough.
   
   There is still a chance that a topic creation passed the check before ns deletion started, and completed after the ns is marked deleted. I don't think we can solve it easily, this will require either distributed locking or serialization of ns/topic metadata operations e.g. via global system 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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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