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

(Blocking)AtomicLock should notifies for state changes. #1071

Closed
franz1981 opened this issue Apr 26, 2020 · 2 comments
Closed

(Blocking)AtomicLock should notifies for state changes. #1071

franz1981 opened this issue Apr 26, 2020 · 2 comments
Labels
archived Archived issues from the legacy Java implementation of Atomix legacy Issues from the legacy Java implementation of Atomix

Comments

@franz1981
Copy link

Expected behavior

BlockingAtomicLock should notifies for state changes.

Actual behavior

BlockingAtomicLock isn't implementing DistributedPrimitive::addStateChangeListener and removeStateChangeListener, hence won't be able to add/remove (and, by consequence, notify) listeners.

Environment

  • Atomix: 3.1.7
@johnou
Copy link
Member

johnou commented Apr 26, 2020

@franz1981 can you provide a pull request with unit test?

@franz1981
Copy link
Author

Sure, will do it tomorrow :)

@kuujo kuujo closed this as completed Jan 13, 2023
@kuujo kuujo added archived Archived issues from the legacy Java implementation of Atomix legacy Issues from the legacy Java implementation of Atomix labels Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
archived Archived issues from the legacy Java implementation of Atomix legacy Issues from the legacy Java implementation of Atomix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@johnou @kuujo @franz1981