Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Separated Statistics [4/7ish]#5891

Closed
reivilibre wants to merge 5 commits into
rei/rss_targetfrom
rei/rss_inc4
Closed

Separated Statistics [4/7ish]#5891
reivilibre wants to merge 5 commits into
rei/rss_targetfrom
rei/rss_inc4

Conversation

@reivilibre
Copy link
Copy Markdown
Contributor

Signed-off-by: Olivier Wilkinson (reivilibre) olivier@librepush.net

This PR is the fourth in a series of PRs replacing #5847, which does the following:

  • Performs old collection when updating stats with deltas (

These PRs will be merged into an intermediate branch (#5879) as some features may be broken if not all the PRs are applied at once.

Note that this may cause DB I/O and CPU spikes at the end of each stats period (e.g. each day), so I probably need to modify this to do less work – how about one batch of 250 rows that must include the one we are trying to update?

Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
Old collection is not included in this commit

Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
@reivilibre reivilibre requested a review from a team August 20, 2019 13:38
Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
@reivilibre
Copy link
Copy Markdown
Contributor Author

Obsolete by changes in #5889

@reivilibre reivilibre closed this Aug 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant