Only fetch metadata for files for a speedup

Byron · May 30, 2021 · d381c6c · d381c6c
1 parent ffdb0c2
commit d381c6c
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/src/aggregate.rs b/src/aggregate.rs
@@ -84,7 +84,7 @@ pub fn aggregate(
                             num_errors += 1;
                             0
                         }
-                        None => unreachable!("must have populated client state for metadata"),
+                        None => 0, // ignore directory
                     } as u128;
                     stats.largest_file_in_bytes = stats.largest_file_in_bytes.max(file_size);
                     stats.smallest_file_in_bytes = stats.smallest_file_in_bytes.min(file_size);

diff --git a/src/common.rs b/src/common.rs
@@ -139,7 +139,9 @@ impl WalkOptions {
             .process_read_dir(|_, _, _, dir_entry_results| {
                 dir_entry_results.iter_mut().for_each(|dir_entry_result| {
                     if let Ok(dir_entry) = dir_entry_result {
-                        dir_entry.client_state = Some(dir_entry.metadata());
+                        if dir_entry.file_type.is_file() {
+                            dir_entry.client_state = Some(dir_entry.metadata());
+                        }
                     }
                 })
             })

diff --git a/src/traverse.rs b/src/traverse.rs
@@ -135,7 +135,7 @@ impl Traversal {
                                 data.metadata_io_error = true;
                                 0
                             }
-                            None => unreachable!("must have populated client state for metadata"),
+                            None => 0, // a directory
                         } as u128;
 
                         match (entry.depth, previous_depth) {