GH-45073: [C++][Parquet] Fix generation of repetition levels for encr…

…yption test data (#45074) ### Rationale for this change This makes the test data readable by other Parquet implementations that validate the repetition levels. ### What changes are included in this PR? * Corrects the generation of encryption test files so that the int64 list columns correctly start lists with repetition level 0. * Updates the parquet-testing submodule to use the corrected files. ### Are these changes tested? Yes, covered by existing tests. ### Are there any user-facing changes? No * GitHub Issue: #45073 Authored-by: Adam Reeve <[email protected]> Signed-off-by: Antoine Pitrou <[email protected]>
apache · Jan 6, 2025 · a931aff · a931aff
1 parent ada8750
commit a931aff
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/cpp/src/parquet/encryption/test_encryption_util.cc b/cpp/src/parquet/encryption/test_encryption_util.cc
@@ -116,9 +116,9 @@ ColumnData<Int64Type> GenerateSampleData<Int64Type>(int rows) {
     int64_t value = i * 1000 * 1000;
     value *= 1000 * 1000;
     int16_t definition_level = 1;
-    int16_t repetition_level = 0;
+    int16_t repetition_level = 1;
     if ((i % 2) == 0) {
-      repetition_level = 1;  // start of a new record
+      repetition_level = 0;  // start of a new record
     }
     int64_col.values.push_back(value);
     int64_col.definition_levels.push_back(definition_level);

diff --git a/cpp/submodules/parquet-testing b/cpp/submodules/parquet-testing
+ −		bad_data/ARROW-RS-GH-6229-DICTHEADER.parquet
+20 −1		data/README.md
+ −		data/alltypes_dictionary.parquet
+ −		data/alltypes_plain.parquet
+ −		data/alltypes_plain.snappy.parquet
+ −		data/encrypt_columns_and_footer.parquet.encrypted
+ −		data/encrypt_columns_and_footer_aad.parquet.encrypted
+ −		data/encrypt_columns_and_footer_ctr.parquet.encrypted
+ −		data/encrypt_columns_and_footer_disable_aad_storage.parquet.encrypted
+ −		data/encrypt_columns_plaintext_footer.parquet.encrypted
+164 −0		data/map_no_value.md
+ −		data/map_no_value.parquet
+ −		data/nan_in_stats.parquet
+ −		data/repeated_primitive_no_list.parquet
+ −		data/uniform_encryption.parquet.encrypted