Skip to content

Commit

Permalink
add todo
Browse files Browse the repository at this point in the history
  • Loading branch information
marvin-j97 committed Jan 14, 2024
1 parent de83bdc commit 61e69ae
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions smoltable/src/table/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -289,6 +289,8 @@ impl Smoltable {
})
} */

// TODO: count thrashes block cache

// TODO: unit test
pub fn count(&self) -> fjall::Result<(usize, usize)> {
use reader::Reader as TableReader;
Expand Down Expand Up @@ -328,6 +330,8 @@ impl Smoltable {
Ok((row_count, cell_count))
}

// TODO: GC thrashes block cache

// TODO: unit test
pub fn run_version_gc(&self) -> fjall::Result<u64> {
use reader::Reader as TableReader;
Expand Down Expand Up @@ -429,6 +433,8 @@ impl Smoltable {
Ok(deleted_cell_count)
}

// TODO: delete row thrashes block cache

// TODO: allow deleting specific columns -> DeleteRowInput, also batch + limit it?
pub fn delete_row(&self, row_key: String) -> fjall::Result<u64> {
let mut count = 0;
Expand Down

0 comments on commit 61e69ae

Please sign in to comment.