Skip to content

Commit

Permalink
chore: add comments
Browse files Browse the repository at this point in the history
  • Loading branch information
WenyXu committed Jan 22, 2025
1 parent 5b9aaa2 commit 3628e54
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/mito2/src/memtable/partition_tree/tree.rs
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ impl PartitionTree {
}

fn verify_primary_key_length(&self, kv: &KeyValue) -> Result<()> {
// The sparse primary key codec does not have a fixed number of fields.
if let Some(expected_num_fields) = self.row_codec.num_fields() {
ensure!(
expected_num_fields == kv.num_primary_keys(),
Expand All @@ -106,6 +107,7 @@ impl PartitionTree {
}
);
}
// TODO(weny): verify the primary key length for sparse primary key codec.
Ok(())
}

Expand Down

0 comments on commit 3628e54

Please sign in to comment.