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

Update clear stats #4324

Merged
merged 3 commits into from
Jan 5, 2024
Merged

Update clear stats #4324

merged 3 commits into from
Jan 5, 2024

Conversation

louislam
Copy link
Owner

@louislam louislam commented Jan 5, 2024

No description provided.

@louislam louislam added this to the 2.0.0 milestone Jan 5, 2024
@louislam louislam mentioned this pull request Jan 5, 2024
17 tasks
@louislam louislam marked this pull request as ready for review January 5, 2024 12:49
@louislam louislam merged commit 9356e7d into master Jan 5, 2024
9 checks passed
@CommanderStorm CommanderStorm deleted the update-clear-stats branch January 5, 2024 12:58
@Saibamen
Copy link
Contributor

Saibamen commented Jan 6, 2024

// Restart all monitors to reset the stats
for (let monitorID in server.monitorList) {
    await restartMonitor(socket.userID, monitorID);
}

This code can also be added to branch 1.23.X

await R.exec("DELETE FROM stat_minutely");

// Restart all monitors to reset the stats
for (let monitorID in server.monitorList) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why we want to start all monitors here? Function restartMonitor is calling startMonitor, so this will also start paused monitors

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 this pull request may close these issues.

2 participants