Skip to content

Commit 835e13a

Browse files
authored
Fix misleading file extension for partition statistics (#3472)
# Rationale for this change The partition statistics file doesn't use Puffin file: * https://iceberg.apache.org/spec/#partition-statistics-file > Statistics information for each unique partition tuple is stored as a row in any of the data file format of the table (for example, Parquet or ORC) ## Are these changes tested? Yes ## Are there any user-facing changes? No <!-- In the case of user-facing changes, please add the changelog label. -->
1 parent 12f7646 commit 835e13a

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

‎tests/table/test_init.py‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1599,7 +1599,7 @@ def test_set_partition_statistics_update(table_v2_with_statistics: Table) -> Non
15991599

16001600
partition_statistics_file = PartitionStatisticsFile(
16011601
snapshot_id=snapshot_id,
1602-
statistics_path="s3://bucket/warehouse/stats.puffin",
1602+
statistics_path="s3://bucket/warehouse/stats.parquet",
16031603
file_size_in_bytes=124,
16041604
)
16051605

@@ -1619,7 +1619,7 @@ def test_set_partition_statistics_update(table_v2_with_statistics: Table) -> Non
16191619
expected = """
16201620
{
16211621
"snapshot-id": 3055729675574597004,
1622-
"statistics-path": "s3://bucket/warehouse/stats.puffin",
1622+
"statistics-path": "s3://bucket/warehouse/stats.parquet",
16231623
"file-size-in-bytes": 124
16241624
}"""
16251625

@@ -1637,7 +1637,7 @@ def test_remove_partition_statistics_update(table_v2_with_statistics: Table) ->
16371637

16381638
partition_statistics_file = PartitionStatisticsFile(
16391639
snapshot_id=snapshot_id,
1640-
statistics_path="s3://bucket/warehouse/stats.puffin",
1640+
statistics_path="s3://bucket/warehouse/stats.parquet",
16411641
file_size_in_bytes=124,
16421642
)
16431643

0 commit comments

Comments
 (0)