-
Notifications
You must be signed in to change notification settings - Fork 1k
Closed
Labels
good first issueGood for newcomersGood for newcomersparquetChanges to the parquet crateChanges to the parquet crate
Description
Unrelated but these public methods on VariantMetadata
are redundant:
arrow-rs/parquet-variant/src/variant/metadata.rs
Lines 210 to 213 in 03a837e
/// The number of metadata dictionary entries | |
pub fn len(&self) -> usize { | |
self.dictionary_size() | |
} |
arrow-rs/parquet-variant/src/variant/metadata.rs
Lines 288 to 291 in 03a837e
/// Get the dictionary size | |
pub const fn dictionary_size(&self) -> usize { | |
self.dictionary_size as _ | |
} |
Originally posted by @friendlymatthew in #7943 (comment)
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomersparquetChanges to the parquet crateChanges to the parquet crate