rust/greptimedb
1
0
Fork 0
mirror of https://github.com/GreptimeTeam/greptimedb.git synced 2026-05-16 13:00:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor: remove double checks of memtable size (#4117)

Browse Source 
* refactor: remove unnecessary unwrap

* Update src/mito2/src/region/version.rs

Co-authored-by: dennis zhuang <killme2008@gmail.com>

---------

Co-authored-by: Kermit <chenjiawei1@xiaohongshu.com>
Co-authored-by: dennis zhuang <killme2008@gmail.com>
This commit is contained in:
cjw
2024-06-08 07:34:03 +08:00
committed by GitHub
parent e142ca40d7
commit d8b51cfaba
1 changed files with 4 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/mito2/src/region/version.rs
View File

@@ -80,14 +80,11 @@ impl VersionControl {
/// Freezes the mutable memtable if it is not empty.
pub(crate) fn freeze_mutable(&self) -> Result<()> {
let version = self.current().version;
if version.memtables.mutable.is_empty() {
let Some(new_memtables) = version.memtables.freeze_mutable(&version.metadata)? else {
return Ok(());
}
// Safety: Immutable memtable is None.
let new_memtables = version
.memtables
.freeze_mutable(&version.metadata)?
.unwrap();
};
// Create a new version with memtable switched.
let new_version = Arc::new(
VersionBuilder::from_version(version)