From 8337c875e6c9b8f24b3131b67511eadf0dc8d4e5 Mon Sep 17 00:00:00 2001 From: Ye Yuan Date: Sun, 18 Aug 2024 22:46:06 +0800 Subject: [PATCH] update comments --- parquet_derive/src/lib.rs | 3 +-- parquet_derive_test/src/lib.rs | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/parquet_derive/src/lib.rs b/parquet_derive/src/lib.rs index 294116a56de2..afbf1c3a7b5e 100644 --- a/parquet_derive/src/lib.rs +++ b/parquet_derive/src/lib.rs @@ -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); } diff --git a/parquet_derive_test/src/lib.rs b/parquet_derive_test/src/lib.rs index ee23d6ce3983..42c4671f4143 100644 --- a/parquet_derive_test/src/lib.rs +++ b/parquet_derive_test/src/lib.rs @@ -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,