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.
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
[fix][meta] Follow up #19817, Fix race condition between ResourceLock update and invalidation #19844
Conversation
… update and invalidation
...ar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/ResourceLockImpl.java
Show resolved
Hide resolved
...ar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/ResourceLockImpl.java
Outdated
Show resolved
Hide resolved
...ar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/ResourceLockImpl.java
Show resolved
Hide resolved
@merlimat @codelipenghui @Jason918 Please take a look. Thanks! |
...ar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/ResourceLockImpl.java
Outdated
Show resolved
Hide resolved
...ar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/ResourceLockImpl.java
Outdated
Show resolved
Hide resolved
...ar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/ResourceLockImpl.java
Show resolved
Hide resolved
The test has passed. And the original test Could you please take a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
...ar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/ResourceLockImpl.java
Show resolved
Hide resolved
...ar-metadata/src/main/java/org/apache/pulsar/metadata/coordination/impl/ResourceLockImpl.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Motivation
This PR is following up the PR #19817. It includes three parts as follows:
revalidate
into two methods to eliminate some flags.Modifications
revalidate
into two methods to eliminate some flags.Verifying this change
Documentation
doc
doc-required
doc-not-needed
doc-complete