chore(parquet): apply suggestions from code review

Co-authored-by: Ed Seidl <etseidl@users.noreply.github.com>

Author: kszucs

parquet/src/arrow/arrow_writer/mod.rs

@@ -266,18 +266,14 @@ impl<W: Write + Send> ArrowWriter<W> {
         let row_group_writer_factory =
             ArrowRowGroupWriterFactory::new(&file_writer, arrow_schema.clone());
 
-        let cdc_chunkers = match props_ptr.content_defined_chunking() {
-            Some(opts) => {
-                let chunkers = file_writer
+        let cdc_chunkers = props_ptr.content_defined_chunking().map(|opts| {
+            file_writer
                     .schema_descr()
                     .columns()
                     .iter()
                     .map(|desc| ContentDefinedChunker::new(desc, opts))
-                    .collect::<Result<Vec<_>>>()?;
-                Some(chunkers)
-            }
-            None => None,
-        };
+                    .collect::<Result<Vec<_>>>()
+        }).transpose()?;
 
         Ok(Self {
             writer: file_writer,

parquet/src/file/properties.rs

@@ -83,10 +83,12 @@ pub struct CdcOptions {
     pub min_chunk_size: usize,
     /// Maximum chunk size in bytes, default is 1024 KiB.
     /// The chunker will create a new chunk whenever the chunk size exceeds this value.
-    /// Note that the parquet writer has a related `data_page_size_limit` property that
+    /// Note that the parquet writer has a related [`data_page_size_limit`] property that
     /// controls the maximum size of a parquet data page after encoding. While setting
     /// `data_page_size_limit` to a smaller value than `max_chunk_size` doesn't affect
     /// the chunking effectiveness, it results in more small parquet data pages.
+    ///
+    /// [`data_page_size_limit`]: WriterPropertiesBuilder::set_data_page_size_limit
     pub max_chunk_size: usize,
     /// Number of bit adjustment to the gearhash mask in order to center the chunk size
     /// around the average size more aggressively, default is 0.