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

Thread safety on MapContainer.addInterceptor #7520

Closed
neilstevenson opened this issue Feb 15, 2016 · 3 comments

Comments

Projects
None yet
3 participants
@neilstevenson
Copy link

commented Feb 15, 2016

The addInterceptor() method in com.hazelcast.map.impl.MapContainer() isn't thread safe.

For example, if two concurrent attempts are made to inject the same interceptor, these will be different interceptor objects with the same id.
In this case, the call to interceptorMap.put(id, interceptor) will increase the map size by one,
but the call to interceptors.add(interceptor) will increase the list size by two.

@jerrinot jerrinot added this to the 3.7 milestone Feb 15, 2016

@jerrinot

This comment has been minimized.

Copy link
Contributor

commented Feb 23, 2016

fixed by #7531

@jerrinot jerrinot closed this Feb 23, 2016

@jerrinot jerrinot reopened this Feb 23, 2016

@jerrinot

This comment has been minimized.

Copy link
Contributor

commented Feb 23, 2016

re-opening until it's confirmed by @ahmetmircik

@ahmetmircik

This comment has been minimized.

Copy link
Member

commented Feb 23, 2016

closed by #7531

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.