Skip to content

Commit e2b7987

Browse files
committed
fix
1 parent 2a60c18 commit e2b7987

File tree

1 file changed

+0
-40
lines changed

1 file changed

+0
-40
lines changed

kernel/src/actions/mod.rs

Lines changed: 0 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -465,46 +465,6 @@ impl Protocol {
465465
.is_some_and(|features| features.contains(feature))
466466
}
467467

468-
/// Checks if a feature is supported in this protocol, accounting for both legacy protocols
469-
/// (where features are implicit based on version) and modern protocols (where features must
470-
/// be in the explicit feature lists).
471-
pub(crate) fn is_feature_supported(&self, feature: &TableFeature) -> bool {
472-
use crate::table_features::FeatureType;
473-
474-
let Some(info) = feature.info() else {
475-
return false;
476-
};
477-
478-
match info.feature_type {
479-
FeatureType::Writer => {
480-
if self.min_writer_version < 7 {
481-
// Legacy writer: check if version meets minimum requirement
482-
self.min_writer_version >= info.min_writer_version
483-
} else {
484-
// Modern protocol: check explicit writer_features list
485-
self.has_table_feature(feature)
486-
}
487-
}
488-
FeatureType::ReaderWriter => {
489-
let reader_supported = if self.min_reader_version < 3 {
490-
self.min_reader_version >= info.min_reader_version
491-
} else {
492-
self.reader_features()
493-
.is_some_and(|features| features.contains(feature))
494-
};
495-
496-
let writer_supported = if self.min_writer_version < 7 {
497-
self.min_writer_version >= info.min_writer_version
498-
} else {
499-
self.has_table_feature(feature)
500-
};
501-
502-
reader_supported && writer_supported
503-
}
504-
FeatureType::Unknown => false,
505-
}
506-
}
507-
508468
/// Validates the relationship between reader features and writer features in the protocol.
509469
pub(crate) fn validate_table_features(&self) -> DeltaResult<()> {
510470
// The protocol states that Reader features may be present if and only if the min_reader_version is 3

0 commit comments

Comments
 (0)