This is an automated email from the ASF dual-hosted git repository.
suxiaogang223 pushed a commit to branch refact_reader_branch
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/refact_reader_branch by this
push:
new d8075142a42 [fix](be) Match parquet list test schema nullability
d8075142a42 is described below
commit d8075142a42e88061204c32a32c1264fb04e7011
Author: Socrates <[email protected]>
AuthorDate: Fri May 29 10:24:45 2026 +0800
[fix](be) Match parquet list test schema nullability
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary: Construct Arrow list arrays in ParquetColumnReaderTest
with explicit element field nullability so the generated arrays match the
declared table schema.
### Release note
None
### Check List (For Author)
- Test: Manual test
- Ran build-support/clang-format.sh on parquet_column_reader_test.cpp.
- Ran git diff --check.
- Fedora ParquetColumnReaderTest is rerun after this fix.
- Behavior changed: No
- Does this need documentation: No
---
be/test/format/new_parquet/parquet_column_reader_test.cpp | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/be/test/format/new_parquet/parquet_column_reader_test.cpp
b/be/test/format/new_parquet/parquet_column_reader_test.cpp
index 059e9b709aa..1bacab7cf8e 100644
--- a/be/test/format/new_parquet/parquet_column_reader_test.cpp
+++ b/be/test/format/new_parquet/parquet_column_reader_test.cpp
@@ -177,7 +177,8 @@ protected:
std::shared_ptr<arrow::Array> build_required_int_list_array() {
auto value_builder = std::make_shared<arrow::Int32Builder>();
- arrow::ListBuilder builder(arrow::default_memory_pool(),
value_builder);
+ arrow::ListBuilder builder(arrow::default_memory_pool(), value_builder,
+ arrow::list(arrow::field("element",
arrow::int32(), false)));
const std::vector<std::vector<int32_t>> values = {
{1, 2}, {3}, {4, 5, 6}, {7}, {8, 9},
};
@@ -192,7 +193,8 @@ protected:
std::shared_ptr<arrow::Array> build_nullable_int_list_array() {
auto value_builder = std::make_shared<arrow::Int32Builder>();
- arrow::ListBuilder builder(arrow::default_memory_pool(),
value_builder);
+ arrow::ListBuilder builder(arrow::default_memory_pool(), value_builder,
+ arrow::list(arrow::field("element",
arrow::int32(), true)));
EXPECT_TRUE(builder.Append().ok());
EXPECT_TRUE(value_builder->Append(10).ok());
EXPECT_TRUE(value_builder->Append(20).ok());
@@ -208,7 +210,8 @@ protected:
std::shared_ptr<arrow::Array> build_required_nullable_int_list_array() {
auto value_builder = std::make_shared<arrow::Int32Builder>();
- arrow::ListBuilder builder(arrow::default_memory_pool(),
value_builder);
+ arrow::ListBuilder builder(arrow::default_memory_pool(), value_builder,
+ arrow::list(arrow::field("element",
arrow::int32(), true)));
EXPECT_TRUE(builder.AppendEmptyValue().ok());
EXPECT_TRUE(builder.Append().ok());
EXPECT_TRUE(value_builder->AppendNull().ok());
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
