Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix CPU 100% when deleting namespace (#10337) #10454

Merged
merged 1 commit into from
May 3, 2021
Merged

Fix CPU 100% when deleting namespace (#10337) #10454

merged 1 commit into from
May 3, 2021

Commits on Apr 30, 2021

  1. Fix CPU 100% when deleting namespace (apache#10337) 

    When deleting the namespace, the namespace Policies will be marked as deleted.
This will trigger topic's `onPoliciesUpdate`
However, in onPoliciesUpdate, the data of the Policies node on zk will be read, such as: `checkReplicationAndRetryOnFailure`
Due to the deletion of the namespace, the zk node may no longer exist at this time.
Failure to read data will trigger infinite retries.
https://github.com/apache/pulsar/blob/e970c2947aff9231202ab72bdbad047d85c55633/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java#L1175-L1193

If there are many topics, there will be a short-term CPU spike

![image](https://user-images.githubusercontent.com/9758905/115834541-ebc32480-a447-11eb-887a-95c4a3d1adf1.png)
    @315157973
    315157973 committed Apr 30, 2021
    Configuration menu
    Copy the full SHA
    f8a06e0 View commit details
    Browse the repository at this point in the history