Skip to content

Commit 998127c

Browse files
authored
core: Remove view cleanup trace logs (#5137)
Those were introduced at info level to debug apparent deadlocking issues, which turned out to be unrelated. They generate quite a bit of noise. Removing them instead of lowering the level as we already have sufficient logging around view cleanup. # Expected complexity level and risk 1 # Testing Logging only.
1 parent 247a9eb commit 998127c

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

crates/core/src/db/relational_db.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ use crate::util::asyncify;
66
use crate::worker_metrics::WORKER_METRICS;
77
use anyhow::{anyhow, Context};
88
use enum_map::EnumMap;
9-
use log::info;
109
use spacetimedb_commitlog::repo::OnNewSegmentFn;
1110
use spacetimedb_commitlog::{self as commitlog, Commitlog, SizeOnDisk};
1211
use spacetimedb_data_structures::map::HashSet;
@@ -1110,15 +1109,13 @@ pub fn spawn_view_cleanup_loop(db: Arc<RelationalDB>) -> tokio::task::AbortHandl
11101109
// in milliseconds, which may be too long for async tasks.
11111110
let db = db.clone();
11121111
let db_identity = db.database_identity();
1113-
info!("running view cleanup for database {db_identity}");
11141112
tokio::task::spawn_blocking(move || run_view_cleanup(&db))
11151113
.await
11161114
.inspect_err(|e| {
11171115
log::error!("[{}] DATABASE: failed to run view cleanup task: {}", db_identity, e);
11181116
})
11191117
.ok();
11201118

1121-
info!("pausing view cleanup for database {db_identity}");
11221119
tokio::time::sleep(VIEWS_EXPIRATION).await;
11231120
}
11241121
})

0 commit comments

Comments
 (0)