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.

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

[TRACKING ISSUE] Remove listeners sync with the calling thread [API-1595] #1016

Open
hz-devops-test opened this issue Aug 9, 2021 · 1 comment
Labels
Milestone

Comments

@hz-devops-test
Copy link

The tracking issue for the Java side PR.

See hazelcast/hazelcast#19289 for details.


It turns out that there are some services relying on the removal
of a listener on the member when listener is removed from the client.
Example:
Cache listener count is kept per proxy. When listener is removed,
we decrease the listener count from the cache context.
Since listener removal is async, it could be the case that the cache
is destroyed from the calling thread before we decrement the value.
In such cases, we decrement but the incremented value is lost. Therefore,
we can end up with negative values.

The listener derregistration invoctions are not waited. It was
sync but we have changed the behavior as a side affect during
this fix hazelcast/hazelcast#17646

We may solve this only for CacheListener but there could be more
services relying on sync listener removal. Therefore, as a fix,
we wait for invocation answer from the remote.

fixes hazelcast/hazelcast#19269

@srknzl srknzl added this to the Backlog milestone Sep 29, 2021
@srknzl srknzl added the to-jira label Sep 29, 2022
@github-actions github-actions bot changed the title [TRACKING ISSUE] Remove listeners sync with the calling thread [TRACKING ISSUE] Remove listeners sync with the calling thread [API-1595] Sep 29, 2022
@github-actions
Copy link

Internal Jira issue: API-1595

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants