Skip to content

Commit

Permalink
update comments
Browse files Browse the repository at this point in the history
  • Loading branch information
double-free committed Aug 18, 2024
1 parent 3590f4e commit 8337c87
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions parquet_derive/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -205,10 +205,9 @@ pub fn parquet_record_reader(input: proc_macro::TokenStream) -> proc_macro::Toke

let mut row_group_reader = row_group_reader;

// build map to index
// key: parquet file column name, value: column index
let mut name_to_index = std::collections::HashMap::new();
for (idx, col) in row_group_reader.metadata().schema_descr().columns().iter().enumerate() {
// println!("col {} name {:?}", idx, col.name());
name_to_index.insert(col.name().to_string(), idx);
}

Expand Down
2 changes: 1 addition & 1 deletion parquet_derive_test/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ struct AnOptionalRecord {

// This struct removes several fields from the "APartiallyCompleteRecord",
// and it shuffles the fields.
// we should still be able to load it from APartiallyCompleteRecord
// we should still be able to load it from APartiallyCompleteRecord parquet file
#[derive(PartialEq, ParquetRecordReader, Debug)]
struct APrunedRecord {
pub bool: bool,
Expand Down

0 comments on commit 8337c87

Please sign in to comment.