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

bad performance Isgranted with redis #7788

Closed
cem1835 opened this issue Feb 20, 2021 · 6 comments
Closed

bad performance Isgranted with redis #7788

cem1835 opened this issue Feb 20, 2021 · 6 comments
Assignees
Milestone

Comments

@cem1835
Copy link

cem1835 commented Feb 20, 2021

Hello,

My application has 500+ Permission Policy and when I use redis, the average page load speed increasing by 50 seconds. (I don't use local redis).

I have attached the log files

I think you can change the caching mechanism on permissions (like ambient context
logs-20210221.txt

Thank you

@maliming maliming added this to the 4.3-preview milestone Feb 21, 2021
@maliming maliming self-assigned this Feb 21, 2021
@leonkosak
Copy link
Contributor

Related?

@maliming
Copy link
Member

maliming commented Mar 2, 2021

Resolved by #7799

@maliming maliming closed this as completed Mar 2, 2021
@cem1835
Copy link
Author

cem1835 commented Aug 18, 2021

problem still going on when loading the /Abp/ApplicationConfigurationScript

i found this while searching for the problem

it loads all the cache one by one

@maliming
Copy link
Member

hi

It checks the permissions one time.

var result = await _permissionChecker.IsGrantedAsync(abpPolicyNames.ToArray());

@cem1835
Copy link
Author

cem1835 commented Aug 18, 2021

permissionchecker performance 47 sec
image

@maliming
Copy link
Member

You can further debug or use tools to check the problematic code that affects performance.

Logger.LogDebug($"PermissionStore.GetCacheItemAsync: {string.Join(",", cacheKeys)}");



protected virtual async Task<byte[][]> GetAndRefreshManyAsync(

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

Successfully merging a pull request may close this issue.

3 participants