chore: remove TODOs

Author: feniljain

crates/iceberg/src/arrow/value.rs

@@ -425,7 +425,7 @@ impl SchemaWithPartnerVisitor<ArrayRef> for ArrowArrayToIcebergStructConverter {
     }
 }
 
-/// TODO(feniljain)
+/// Partner type representing accessing and walking arrow arrays alongside iceberg schema
 pub struct ArrowArrayAccessor;
 
 impl PartnerAccessor<ArrayRef> for ArrowArrayAccessor {

crates/iceberg/src/writer/file_writer/parquet_writer.rs

@@ -317,8 +317,6 @@ impl MinMaxColAggregator {
     }
 }
 
-// TODO(feniljain): Think do we need to add nan_value_counts to `parquet_files_to_data_files` and
-// `parquet_to_data_file_builder` too?
 impl ParquetWriter {
     /// Converts parquet files to data files
     #[allow(dead_code)]
@@ -525,7 +523,6 @@ impl FileWriter for ParquetWriter {
 
         self.current_row_num += batch.num_rows();
 
-        // TODO(feniljain): Confirm if this `clone` is okay to perform
         let batch_c = batch.clone();
         self.nan_value_count_visitor.compute(self.schema.clone(), batch_c)?;