This is an automated email from the ASF dual-hosted git repository.

alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git


The following commit(s) were added to refs/heads/main by this push:
     new 70ae764c03 minor: Re-enable CDC bench (#9686)
70ae764c03 is described below

commit 70ae764c03b8f5dd10a9ec109c9c0a446a81e2b2
Author: Ed Seidl <[email protected]>
AuthorDate: Fri Apr 10 10:56:10 2026 -0700

    minor: Re-enable CDC bench (#9686)
    
    # Which issue does this PR close?
    
    N/A
    
    # Rationale for this change
    
    The content defined chunking bench (CDC) was disabled due to a bug
    (#9637). That has since been fixed (#9644), so the bench can be
    re-enabled.
    
    # What changes are included in this PR?
    
    Uncomments out the relevant bench in arrow_writer
    
    # Are these changes tested?
    
    N/A
    
    # Are there any user-facing changes?
    
    No
---
 parquet/benches/arrow_writer.rs | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/parquet/benches/arrow_writer.rs b/parquet/benches/arrow_writer.rs
index 6b48afbf3d..9b22bb04b3 100644
--- a/parquet/benches/arrow_writer.rs
+++ b/parquet/benches/arrow_writer.rs
@@ -34,7 +34,7 @@ use arrow::util::bench_util::{create_f16_array, 
create_f32_array, create_f64_arr
 use arrow::{record_batch::RecordBatch, util::data_gen::*};
 use arrow_array::RecordBatchOptions;
 use parquet::errors::Result;
-use parquet::file::properties::{WriterProperties, WriterVersion};
+use parquet::file::properties::{CdcOptions, WriterProperties, WriterVersion};
 
 fn create_primitive_bench_batch(
     size: usize,
@@ -457,12 +457,10 @@ fn create_writer_props() -> Vec<(&'static str, 
WriterProperties)> {
         .build();
     props.push(("zstd_parquet_2", prop));
 
-    // Disabled until https://github.com/apache/arrow-rs/issues/9637 is fixed
-    //
-    // let prop = WriterProperties::builder()
-    //    .set_content_defined_chunking(Some(CdcOptions::default()))
-    //    .build();
-    // props.push(("cdc", prop));
+    let prop = WriterProperties::builder()
+        .set_content_defined_chunking(Some(CdcOptions::default()))
+        .build();
+    props.push(("cdc", prop));
 
     props
 }

Reply via email to