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

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


The following commit(s) were added to refs/heads/main by this push:
     new f7506497c9 GH-34446: [C++][Parquet] Fix RecordReaderPrimitveTypeTests 
test (#34447)
f7506497c9 is described below

commit f7506497c94b2429fcfb557d03b4b453bf564c7b
Author: Will Jones <[email protected]>
AuthorDate: Fri Mar 3 15:35:59 2023 -0800

    GH-34446: [C++][Parquet] Fix RecordReaderPrimitveTypeTests test (#34447)
    
    ### Rationale for this change
    
    The test data was updated in https://github.com/apache/arrow/pull/17877 to 
have a depth of 3.
    
    An assertion was added concurrently to assert the max depth in 
https://github.com/apache/arrow/pull/34318.
    
    This PR fixes the test so the changes are compatible.
    
    ### What changes are included in this PR?
    
    ### Are these changes tested?
    
    ### Are there any user-facing changes?
    
    * Closes: #34446
    
    Authored-by: Will Jones <[email protected]>
    Signed-off-by: Jacob Wujciak-Jens <[email protected]>
---
 cpp/src/parquet/column_reader_test.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/parquet/column_reader_test.cc 
b/cpp/src/parquet/column_reader_test.cc
index 414d79b8fb..2cb6aaa04d 100644
--- a/cpp/src/parquet/column_reader_test.cc
+++ b/cpp/src/parquet/column_reader_test.cc
@@ -936,7 +936,7 @@ TEST_P(RecordReaderPrimitiveTypeTest, ReadNullableRepeated) 
{
   ASSERT_EQ(records_read, 0);
 
   // Test the descr() accessor.
-  ASSERT_EQ(record_reader_->descr()->max_definition_level(), 1);
+  ASSERT_EQ(record_reader_->descr()->max_definition_level(), 3);
 
   // Read [10], null
   // We do not read this null for both reading dense and spaced.

Reply via email to