remove stray TODOs

apache · May 11, 2021 · a9fb246 · a9fb246
1 parent 2894b3f
commit a9fb246
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/parquet/src/arrow/arrow_writer.rs b/parquet/src/arrow/arrow_writer.rs
@@ -1366,7 +1366,7 @@ mod tests {
         let a_list_data = ArrayData::builder(DataType::List(Box::new(Field::new(
             "item",
             DataType::Int32,
-            true, // TODO: why does this fail when false? Is it related to logical nulls?
+            false,
         ))))
         .len(5)
         .add_buffer(a_value_offsets)

diff --git a/parquet/src/arrow/levels.rs b/parquet/src/arrow/levels.rs
@@ -331,7 +331,6 @@ impl LevelInfo {
                 unreachable!("Cannot have a root as a child")
             }
             (LevelType::List(_), _) => {
-                // Always add 1  (TDDO: document why)
                 self.max_definition + 1 + level_type.level_increment()
             }
             (LevelType::Struct(_), _) => {