Skip to content

Commit e772ed8

Browse files
committed
test(parquet): drop redundant large-list schema-evolution test
The test referenced a helper (ValidateParquetSchemaEvolution) that does not exist, so it never compiled. Its large_list->list acceptance is already covered end-to-end by ProjectLargeListType via the same ValidateParquetTypeCompatibility path.
1 parent 7fa1c4d commit e772ed8

1 file changed

Lines changed: 0 additions & 10 deletions

File tree

src/iceberg/test/parquet_schema_test.cc

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -451,16 +451,6 @@ TEST(ParquetGeospatialSchemaTest, RejectsMissingLogicalType) {
451451
HasErrorMessage("Iceberg geometry requires Parquet Geometry logical type"));
452452
}
453453

454-
TEST(ParquetSchemaProjectionTest, ValidateSchemaEvolutionAllowsLargeList) {
455-
::parquet::arrow::SchemaField parquet_field;
456-
parquet_field.field = ::arrow::field("numbers", ::arrow::large_list(::arrow::int32()));
457-
458-
ListType expected_type(
459-
SchemaField::MakeOptional(/*field_id=*/101, "element", iceberg::int32()));
460-
auto status = ValidateParquetSchemaEvolution(expected_type, parquet_field);
461-
ASSERT_THAT(status, IsOk());
462-
}
463-
464454
TEST(ParquetSchemaProjectionTest, ProjectNullPhysicalFieldsAsNull) {
465455
Schema expected_schema({
466456
SchemaField::MakeOptional(/*field_id=*/1, "age", iceberg::int32()),

0 commit comments

Comments
 (0)