(arrow-rs) branch main updated: feat(parquet): derive `PartialEq` and `Eq` for `CdcOptions` (#9602)

[alamb]

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 f912311607 feat(parquet): derive `PartialEq` and `Eq` for `CdcOptions` 
(#9602)
f912311607 is described below

commit f91231160716d2b726a6bd01ef1b596c9ff69e17
Author: Krisztián Szűcs <szucs.kriszt...@gmail.com>
AuthorDate: Tue Mar 31 21:31:35 2026 +0200

    feat(parquet): derive `PartialEq` and `Eq` for `CdcOptions` (#9602)
    
    # Rationale for this change
    
    CdcOptions only contains primitive fields (usize, usize, i32) so
    deriving PartialEq and Eq is straightforward. This is needed by
    downstream crates such as DataFusion that embed CdcOptions in their own
    configuration structs and need to compare them.
    
    # What changes are included in this PR?
    
    Implemented PartialEq and Eq for CdcOptions.
    
    # Are these changes tested?
    
    Added an equality test.
    
    # Are there any user-facing changes?
    
    No.
---
 parquet/src/file/properties.rs | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/parquet/src/file/properties.rs b/parquet/src/file/properties.rs
index ae15cc6b82..640a7a075d 100644
--- a/parquet/src/file/properties.rs
+++ b/parquet/src/file/properties.rs
@@ -80,7 +80,7 @@ pub const DEFAULT_CDC_NORM_LEVEL: i32 = 0;
 /// following options control the chunks' size and the chunking process. Note
 /// that the chunk size is calculated based on the logical value of the data,
 /// before any encoding or compression is applied.
-#[derive(Debug, Clone, Copy)]
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub struct CdcOptions {
     /// Minimum chunk size in bytes, default is 256 KiB.
     /// The rolling hash will not be updated until this size is reached for 
each chunk.
@@ -1864,4 +1864,18 @@ mod tests {
             }
         }
     }
+
+    #[test]
+    fn test_cdc_options_equality() {
+        let opts = CdcOptions::default();
+        assert_eq!(opts, CdcOptions::default());
+
+        let custom = CdcOptions {
+            min_chunk_size: 1024,
+            max_chunk_size: 8192,
+            norm_level: 1,
+        };
+        assert_eq!(custom, custom);
+        assert_ne!(opts, custom);
+    }
 }