Takes inner HashSet when dropping CacheHashData (#33169)

solana-labs · Sep 6, 2023 · a3dc3eb · a3dc3eb
1 parent a80819b
commit a3dc3eb
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/accounts-db/src/cache_hash_data.rs b/accounts-db/src/cache_hash_data.rs
@@ -226,12 +226,12 @@ impl CacheHashData {
         result
     }
     fn delete_old_cache_files(&self) {
-        let pre_existing_cache_files = self.pre_existing_cache_files.lock().unwrap();
-        if !pre_existing_cache_files.is_empty() {
+        let old_cache_files = std::mem::take(&mut *self.pre_existing_cache_files.lock().unwrap());
+        if !old_cache_files.is_empty() {
             self.stats
                 .unused_cache_files
-                .fetch_add(pre_existing_cache_files.len(), Ordering::Relaxed);
-            for file_name in pre_existing_cache_files.iter() {
+                .fetch_add(old_cache_files.len(), Ordering::Relaxed);
+            for file_name in old_cache_files.iter() {
                 let result = self.cache_dir.join(file_name);
                 let _ = fs::remove_file(result);
             }