Skip to content

Commit

Permalink
make persistent caching not printing so much
Browse files Browse the repository at this point in the history
  • Loading branch information
sokra committed Oct 10, 2024
1 parent d0ac6f6 commit 211032d
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs
Original file line number Diff line number Diff line change
Expand Up @@ -248,13 +248,6 @@ impl BackingStorage for LmdbBackingStorage {
meta_updates: Vec<ChunkedVec<CachedDataUpdate>>,
data_updates: Vec<ChunkedVec<CachedDataUpdate>>,
) -> Result<()> {
println!(
"Persisting {} operations, {} task cache updates, {} meta updates, {} data updates...",
operations.len(),
task_cache_updates.iter().map(|u| u.len()).sum::<usize>(),
meta_updates.iter().map(|u| u.len()).sum::<usize>(),
data_updates.iter().map(|u| u.len()).sum::<usize>()
);
let start = Instant::now();

Check warning on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / build / build

unused variable: `start`

Check warning on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / build-native / build

unused variable: `start`

Check warning on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / test unit (18) / build

unused variable: `start`

Check warning on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / test unit (20) / build

unused variable: `start`

Check warning on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / rust check / build

unused variable: `start`

Check failure on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / rust check / build

unused variable: `start`

Check warning on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / rustdoc check / build

unused variable: `start`

Check warning on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / types and precompiled / build

unused variable: `start`

Check warning on line 251 in turbopack/crates/turbo-tasks-backend/src/lmdb_backing_storage.rs

View workflow job for this annotation

GitHub Actions / lint / build

unused variable: `start`
let mut op_count = 0;
let mut tx = self.env.begin_rw_txn()?;
Expand Down Expand Up @@ -417,10 +410,6 @@ impl BackingStorage for LmdbBackingStorage {
tx.commit()
.with_context(|| anyhow!("Unable to commit operations"))?;
}
println!(
"Persisted {op_count} db entries after {:?}",
start.elapsed()
);
Ok(())
}

Expand Down Expand Up @@ -626,6 +615,7 @@ fn serialize_task_data(
let mut serializer = symbol_map.serializer_for(&mut buf).unwrap();
if let Err(err) = serde_path_to_error::serialize(item, &mut serializer) {
if item.is_optional() {
#[cfg(feature = "verify_serialization")]
println!("Skipping non-serializable optional item: {item:?}");
} else {
error = Err(err).context({
Expand Down

0 comments on commit 211032d

Please sign in to comment.