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.