Skip to content

Commit f33754d

Browse files
authored
chore: remove unnecessary manifest reader/writer test cases (#377)
The manifest_writer_versions_test.cc and manifest_list_versions_test.cc should be adequate to cover the manifest module. Remove manifest_list_reader_writer_test.cc and manifest_reader_writer_test.cc along with their related .avro files, so we don't need to ship binary files any more.
1 parent e0b35ef commit f33754d

11 files changed

+0
-728
lines changed

dev/release/rat_exclude_files.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,5 @@ dist/**
2323
.git/**
2424
requirements.txt
2525
test/resources/**
26-
*.avro
2726
*.json
28-
*.parquet
2927
src/iceberg/util/murmurhash3_internal.*

src/iceberg/test/CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,9 +126,7 @@ if(ICEBERG_BUILD_BUNDLE)
126126
avro_test.cc
127127
avro_schema_test.cc
128128
avro_stream_test.cc
129-
manifest_list_reader_writer_test.cc
130129
manifest_list_versions_test.cc
131-
manifest_reader_writer_test.cc
132130
manifest_writer_versions_test.cc
133131
test_common.cc)
134132

src/iceberg/test/manifest_list_reader_writer_test.cc

Lines changed: 0 additions & 391 deletions
This file was deleted.

src/iceberg/test/manifest_reader_writer_test.cc

Lines changed: 0 additions & 333 deletions
This file was deleted.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.

0 commit comments

Comments
 (0)