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

Add ability to recalculate user statistics #3757

Closed
BrokenEagle opened this issue Jun 19, 2018 · 4 comments
Closed

Add ability to recalculate user statistics #3757

BrokenEagle opened this issue Jun 19, 2018 · 4 comments

Comments

@BrokenEagle
Copy link
Collaborator

BrokenEagle commented Jun 19, 2018

Related to #3756. By now, there are probably plenty of user's whose upload counts are off. Actually, this was probably an issue even before asynchronous uploads, as my own uploads are off, and I haven't had any of those failed uploads stuck in the preprocessed state.

However, it's not just uploads, as other statistics are off as well.

Note versions:

Post versions:

Most of the other ones were spot on, but some of them were still off by one.

I see a couple possible solutions to this:

1. Run a daily, weekly, or monthly task that recalculates all statistics for all users

This would be quite laborious as there are over half a million users.

2. Run a recheck everytime a profile page is visited

Not as laborious as 1, although it would still be a bit wasteful unless it was restricted to running once per day, or something like that.

3. Have an on demand recheck option for certain levels of users

Kind of like how the tag fix option works. It could be restricted to just Builder+ to prevent abuse.

@r888888888
Copy link
Collaborator

I just noticed there's a counter_cache on the uploader relation on Post. Is it possible it was double counting all this time? Anyway I don't think a manual increment is necessary.

@evazion
Copy link
Member

evazion commented Jun 20, 2018

@r888888888 2866335 broke the site map. There's a typo in the route, should be new_maintenance_user_count_fixes_path.

@evazion evazion reopened this Jun 20, 2018
@BrokenEagle
Copy link
Collaborator Author

As noted by myself and several others, the refresh function doesn't seem to change any of the numbers for uploads, note changes, or post changes.

@r888888888
Copy link
Collaborator

This is fixed now.

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

No branches or pull requests

3 participants