fedi/lemmy
1
0
Fork 0
mirror of https://github.com/LemmyNet/lemmy.git synced 2024-11-09 09:52:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

Reindex tables non-concurrently on startup. Fixes #1726 . #1723

Browse source
This commit is contained in:
Dessalines 2021-08-25 12:10:56 -04:00
parent e413350be3
commit bcd187cf08
2 changed files with 11 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
crates/utils/src/rate_limit/mod.rs
View file

@ -71,9 +71,7 @@ impl RateLimited {
{
// Does not need to be blocking because the RwLock in settings never held across await points,
// and the operation here locks only long enough to clone
let rate_limit: RateLimitConfig = Settings::get()
.rate_limit
.unwrap_or_else(RateLimitConfig::default);
let rate_limit: RateLimitConfig = Settings::get().rate_limit.unwrap_or_default();
// before
{

17
src/scheduled_tasks.rs
View file

@ -13,10 +13,12 @@ pub fn setup(pool: DbPool) {
let conn = pool.get().unwrap();
active_counts(&conn);
reindex_aggregates_tables(&conn);
// On startup, reindex the tables non-concurrently
reindex_aggregates_tables(&conn, false);
scheduler.every(1.hour()).run(move || {
active_counts(&conn);
reindex_aggregates_tables(&conn);
reindex_aggregates_tables(&conn, true);
});
let conn = pool.get().unwrap();
@ -35,19 +37,20 @@ pub fn setup(pool: DbPool) {
/// 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: &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);
fn reindex_table(conn: &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.");
}