Skip to content

Commit 02979ce

Browse files
authored
Remove redundant cache lookup debug logging. (#12395)
The cache lookup workunit (a few lines up) already mutates itself to report the hit, so additionally reporting the hit on the parent workunit is redundant, and looks like: ``` 15:30:10.02 [DEBUG] Completed: Hit: Local cache lookup: Determine Python imports for src/python/pants/engine/internals/mapper_test.py:tests 15:30:10.03 [DEBUG] Completed: Hit local cache: Scheduling: Determine Python imports for src/python/pants/engine/internals/mapper_test.py:tests ```
1 parent fa490f9 commit 02979ce

File tree

1 file changed

+0
-7
lines changed
  • src/rust/engine/process_execution/src

1 file changed

+0
-7
lines changed

src/rust/engine/process_execution/src/cache.rs

-7
Original file line numberDiff line numberDiff line change
@@ -125,13 +125,6 @@ impl crate::CommandRunner for CommandRunner {
125125
.await;
126126

127127
if let Ok(result) = cache_read_result {
128-
workunit.update_metadata(|initial| WorkunitMetadata {
129-
desc: initial
130-
.desc
131-
.as_ref()
132-
.map(|desc| format!("Hit local cache: {}", desc)),
133-
..initial
134-
});
135128
return Ok(result);
136129
}
137130

0 commit comments

Comments
 (0)