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

Change permission DB storage #21

Open
Maxopoly opened this issue May 6, 2020 · 0 comments
Open

Change permission DB storage #21

Maxopoly opened this issue May 6, 2020 · 0 comments

Comments

@Maxopoly
Copy link
Member

Maxopoly commented May 6, 2020

Deleting rows out of indexed tables causes rows to be reordered, which is really slow for production servers (Civclassic has 4.2 million entries in the permissionByGroup table).

Thus editing perms is laggy, even if the db query is done async it sometimes times out. Every second change is a deletion from permissionByGroup.

A faster way would be to store a boolean flag for every perm, for every rank for every group, update that as needed and bulk delete only if groups are deleted. This would bloat the table size, but to an acceptable degree in comparison to the current performance issues

@Maxopoly Maxopoly added this to Active To Dos - Unclaimed tasks looking for someone to fix them in Public Issue Board May 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Public Issue Board
Active To Dos - Unclaimed tasks looki...
Development

No branches or pull requests

1 participant