use clokwerk::{Scheduler, TimeUnits};
// Import week days and WeekDay
use diesel::{sql_query, PgConnection, RunQueryDsl};
-use lemmy_db_queries::{source::activity::Activity_, DbPool};
-use lemmy_db_schema::source::activity::Activity;
-use log::info;
+use lemmy_db_schema::{source::activity::Activity, utils::DbPool};
+use lemmy_utils::error::LemmyError;
use std::{thread, time::Duration};
+use tracing::info;
/// Schedules various cleanup tasks for lemmy in a background thread
-pub fn setup(pool: DbPool) {
+pub fn setup(pool: DbPool) -> Result<(), LemmyError> {
let mut scheduler = Scheduler::new();
- let conn = pool.get().unwrap();
- active_counts(&conn);
- reindex_aggregates_tables(&conn);
+ let mut conn = pool.get()?;
+ active_counts(&mut conn);
+ update_banned_when_expired(&mut conn);
+
+ // On startup, reindex the tables non-concurrently
+ // TODO remove this for now, since it slows down startup a lot on lemmy.ml
+ reindex_aggregates_tables(&mut conn, true);
scheduler.every(1.hour()).run(move || {
- active_counts(&conn);
- reindex_aggregates_tables(&conn);
+ active_counts(&mut conn);
+ update_banned_when_expired(&mut conn);
+ reindex_aggregates_tables(&mut conn, true);
+ drop_ccnew_indexes(&mut conn);
});
- let conn = pool.get().unwrap();
- clear_old_activities(&conn);
+ let mut conn = pool.get()?;
+ clear_old_activities(&mut conn);
scheduler.every(1.weeks()).run(move || {
- clear_old_activities(&conn);
+ clear_old_activities(&mut conn);
});
// Manually run the scheduler in an event loop
/// Reindex the aggregates tables every one hour
/// This is necessary because hot_rank is actually a mutable function:
/// https://dba.stackexchange.com/questions/284052/how-to-create-an-index-based-on-a-time-based-function-in-postgres?noredirect=1#comment555727_284052
-fn reindex_aggregates_tables(conn: &PgConnection) {
+fn reindex_aggregates_tables(conn: &mut PgConnection, concurrently: bool) {
for table_name in &[
"post_aggregates",
"comment_aggregates",
"community_aggregates",
] {
- reindex_table(&conn, &table_name);
+ reindex_table(conn, table_name, concurrently);
}
}
-fn reindex_table(conn: &PgConnection, table_name: &str) {
- info!("Reindexing table {} ...", table_name);
- let query = format!("reindex table concurrently {}", table_name);
- sql_query(query).execute(conn).unwrap();
+fn reindex_table(conn: &mut PgConnection, table_name: &str, concurrently: bool) {
+ let concurrently_str = if concurrently { "concurrently" } else { "" };
+ info!("Reindexing table {} {} ...", concurrently_str, table_name);
+ let query = format!("reindex table {} {}", concurrently_str, table_name);
+ sql_query(query).execute(conn).expect("reindex table");
info!("Done.");
}
/// Clear old activities (this table gets very large)
-fn clear_old_activities(conn: &PgConnection) {
+fn clear_old_activities(conn: &mut PgConnection) {
info!("Clearing old activities...");
- Activity::delete_olds(&conn).unwrap();
+ Activity::delete_olds(conn).expect("clear old activities");
info!("Done.");
}
/// Re-calculate the site and community active counts every 12 hours
-fn active_counts(conn: &PgConnection) {
+fn active_counts(conn: &mut PgConnection) {
info!("Updating active site and community aggregates ...");
let intervals = vec![
"update site_aggregates set users_active_{} = (select * from site_aggregates_activity('{}'))",
i.1, i.0
);
- sql_query(update_site_stmt).execute(conn).unwrap();
+ sql_query(update_site_stmt)
+ .execute(conn)
+ .expect("update site stats");
let update_community_stmt = format!("update community_aggregates ca set users_active_{} = mv.count_ from community_aggregates_activity('{}') mv where ca.community_id = mv.community_id_", i.1, i.0);
- sql_query(update_community_stmt).execute(conn).unwrap();
+ sql_query(update_community_stmt)
+ .execute(conn)
+ .expect("update community stats");
}
info!("Done.");
}
+
+/// Set banned to false after ban expires
+fn update_banned_when_expired(conn: &mut PgConnection) {
+ info!("Updating banned column if it expires ...");
+ let update_ban_expires_stmt =
+ "update person set banned = false where banned = true and ban_expires < now()";
+ sql_query(update_ban_expires_stmt)
+ .execute(conn)
+ .expect("update banned when expires");
+}
+
+/// Drops the phantom CCNEW indexes created by postgres
+/// https://github.com/LemmyNet/lemmy/issues/2431
+fn drop_ccnew_indexes(conn: &mut PgConnection) {
+ info!("Dropping phantom ccnew indexes...");
+ let drop_stmt = "select drop_ccnew_indexes()";
+ sql_query(drop_stmt)
+ .execute(conn)
+ .expect("drop ccnew indexes");
+}