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

[security] Eliminate synchronization when checking task permissions #10833

Closed
vbekiaris opened this issue Jun 26, 2017 · 1 comment
Closed

[security] Eliminate synchronization when checking task permissions #10833

vbekiaris opened this issue Jun 26, 2017 · 1 comment

Comments

@vbekiaris
Copy link
Contributor

@vbekiaris vbekiaris commented Jun 26, 2017

#10832 & hazelcast/hazelcast-enterprise#1559 introduce synchronization when checking permissions for each message task with security enabled.

@vbekiaris vbekiaris added this to the 3.9 milestone Jun 26, 2017
@mmedenjak mmedenjak changed the title Eliminate synchronization when checking task permissions [security] Eliminate synchronization when checking task permissions Jul 11, 2017
@gurbuzali
Copy link
Member

@gurbuzali gurbuzali commented Jul 25, 2017

I believe this is not a critical issue, because the introduced synchronization is not in the happy-path.
We touch the synchronization block once only for each principal, not for each message task.

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

Successfully merging a pull request may close this issue.

None yet
3 participants