Skip to content

Commit

Permalink
Smaller scope for lock.
Browse files Browse the repository at this point in the history
Signed-off-by: Alex Feinberg <alex@strlen.net>
  • Loading branch information
afeinberg committed Jan 30, 2024
1 parent 49fb3c0 commit 7e38ce2
Showing 1 changed file with 10 additions and 8 deletions.
18 changes: 10 additions & 8 deletions components/region_cache_memory_engine/src/write_batch.rs
Original file line number Diff line number Diff line change
Expand Up @@ -66,14 +66,16 @@ impl RangeCacheWriteBatch {
}

fn write_impl(&mut self, seq: u64) -> Result<()> {
let core = self.engine.core().read().unwrap();
let (engine, filtered_keys) = (
core.engine().clone(),
self.buffer
.iter()
.filter(|&e| e.should_write_to_memory(core.range_manager()))
.collect::<Vec<_>>(),
);
let (engine, filtered_keys) = {
let core = self.engine.core().read().unwrap();
(
core.engine().clone(),
self.buffer
.iter()
.filter(|&e| e.should_write_to_memory(core.range_manager()))
.collect::<Vec<_>>(),
)
};
filtered_keys
.into_iter()
.try_for_each(|e| e.write_to_memory(&engine, seq))
Expand Down

0 comments on commit 7e38ce2

Please sign in to comment.