Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions crates/core/src/kernel/snapshot/iterators.rs
Original file line number Diff line number Diff line change
Expand Up @@ -241,6 +241,10 @@ impl LogicalFileView {
.map(|s| round_ms_datetimes(s, &ceil_datetime))
}

Copy link

Copilot AI Aug 31, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new public method deletion_vector_descriptor lacks documentation. Consider adding a doc comment explaining what the method returns and when it would be None.

Copilot uses AI. Check for mistakes.
pub fn deletion_vector_descriptor(&self) -> Option<DeletionVectorDescriptor> {
self.deletion_vector().map(|dv| dv.descriptor())
}

/// Returns a view into the deletion vector for this file, if present.
fn deletion_vector(&self) -> Option<DeletionVectorView<'_>> {
let dv_col = self
Expand Down