Skip to content

Commit bf64a12

Browse files
committed
refactor: initialize additional fields in DataFile structure for aggregate tests
1 parent 2fe0e90 commit bf64a12

File tree

1 file changed

+18
-5
lines changed

1 file changed

+18
-5
lines changed

src/iceberg/test/aggregate_test.cc

Lines changed: 18 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -442,11 +442,24 @@ TEST(AggregateTest, DataFileAggregatorParity) {
442442
.content_size_in_bytes = std::nullopt,
443443
.partition_spec_id = std::nullopt};
444444

445-
DataFile missing_some_stats{
446-
.file_path = "file_missing_stats.avro",
447-
.record_count = 20,
448-
.value_counts = {{1, 20}, {4, 10}},
449-
};
445+
DataFile missing_some_stats{.file_path = "file_missing_stats.avro",
446+
.partition = {},
447+
.record_count = 20,
448+
.column_sizes = {},
449+
.value_counts = {{1, 20}, {4, 10}},
450+
.null_value_counts = {},
451+
.nan_value_counts = {},
452+
.lower_bounds = {},
453+
.upper_bounds = {},
454+
.key_metadata = {},
455+
.split_offsets = {},
456+
.equality_ids = {},
457+
.sort_order_id = std::nullopt,
458+
.first_row_id = std::nullopt,
459+
.referenced_data_file = std::nullopt,
460+
.content_offset = std::nullopt,
461+
.content_size_in_bytes = std::nullopt,
462+
.partition_spec_id = std::nullopt};
450463
auto [b4_lower, b4_upper] = make_bounds(1, -3, 1333);
451464
missing_some_stats.lower_bounds = std::move(b4_lower);
452465
missing_some_stats.upper_bounds = std::move(b4_upper);

0 commit comments

Comments
 (0)