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

to-public-array
Dessalines 2021-08-26 07:49:16 -04:00 committed by GitHub
parent da5fdae006
commit 0a91a28b27
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 7 deletions

View File

@ -13,10 +13,12 @@ pub fn setup(pool: DbPool) {
let conn = pool.get().unwrap(); let conn = pool.get().unwrap();
active_counts(&conn); 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 || { scheduler.every(1.hour()).run(move || {
active_counts(&conn); active_counts(&conn);
reindex_aggregates_tables(&conn); reindex_aggregates_tables(&conn, true);
}); });
let conn = pool.get().unwrap(); let conn = pool.get().unwrap();
@ -35,19 +37,20 @@ pub fn setup(pool: DbPool) {
/// Reindex the aggregates tables every one hour /// Reindex the aggregates tables every one hour
/// This is necessary because hot_rank is actually a mutable function: /// 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 /// 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 &[ for table_name in &[
"post_aggregates", "post_aggregates",
"comment_aggregates", "comment_aggregates",
"community_aggregates", "community_aggregates",
] { ] {
reindex_table(conn, table_name); reindex_table(conn, table_name, concurrently);
} }
} }
fn reindex_table(conn: &PgConnection, table_name: &str) { fn reindex_table(conn: &PgConnection, table_name: &str, concurrently: bool) {
info!("Reindexing table {} ...", table_name); let concurrently_str = if concurrently { "concurrently" } else { "" };
let query = format!("reindex table concurrently {}", table_name); info!("Reindexing table {} {} ...", concurrently_str, table_name);
let query = format!("reindex table {} {}", concurrently_str, table_name);
sql_query(query).execute(conn).expect("reindex table"); sql_query(query).execute(conn).expect("reindex table");
info!("Done."); info!("Done.");
} }