Skip to content

Commit

Permalink
reword append-only
Browse files Browse the repository at this point in the history
  • Loading branch information
OussamaSaoudi committed Jan 28, 2025
1 parent ff33894 commit f8eded9
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions kernel/src/table_configuration.rs
Original file line number Diff line number Diff line change
Expand Up @@ -192,18 +192,18 @@ impl TableConfiguration {
.enable_deletion_vectors
.unwrap_or(false)
}

/// Returns `true` if the table supports the appendOnly table feature. To support this feature:
/// - The table must have a writer version between 2 and 7 (inclusive)
/// - If the table is on writer version 7, it must have the [`WriterFeatures::AppendOnly`]
/// writer feature.
#[allow(unused)]
#[cfg_attr(feature = "developer-visibility", visibility::make(pub))]
pub(crate) fn is_append_only_supported(&self) -> bool {
match self.protocol.min_writer_version() {
7 if self
.protocol
.has_writer_feature(&WriterFeatures::AppendOnly) =>
{
true
}
ver if (2..7).contains(&ver) => true,
_ => false,
let protocol = &self.protocol;
match protocol.min_writer_version() {
7 if protocol.has_writer_feature(&WriterFeatures::AppendOnly) => true,
version => (2..=6).contains(&version),
}
}

Expand Down

0 comments on commit f8eded9

Please sign in to comment.