-
Notifications
You must be signed in to change notification settings - Fork 1.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