Skip to content

fix: Fix delete partition for transformed partition types#94

Open
xieandrew wants to merge 1 commit intomasterfrom
78-delete-partition-date
Open

fix: Fix delete partition for transformed partition types#94
xieandrew wants to merge 1 commit intomasterfrom
78-delete-partition-date

Conversation

@xieandrew
Copy link
Collaborator

@xieandrew xieandrew commented Jan 22, 2026

Fixes #78 by reading the manifest for transformed partition names instead of column names. It also applies the partition transform to the input if it is a timestamp in order to convert to the Iceberg int representation for time units (e.g. months since epoch).

Examples:

Partitioned by day:
ice delete nyc.taxis_p_by_day --partition '[{"name": "tpep_pickup_datetime_day", "values": ["2024-12-31"]}]'

Partitioned by month:
ice delete nyc.taxis_p_by_month --partition '[{"name": "tpep_pickup_datetime_month", "values": ["2025-02-01"]}]'

Partitioned by hour:
ice delete nyc.taxis_p_by_hour --partition '[{"name": "tpep_pickup_datetime_hour", "values": ["2025-01-01T03:00:00"]}]'

@xieandrew xieandrew force-pushed the 78-delete-partition-date branch from 925fbac to b7fd9b8 Compare February 3, 2026 17:35
@xieandrew xieandrew marked this pull request as ready for review February 3, 2026 17:45
@xieandrew xieandrew force-pushed the 78-delete-partition-date branch from b7fd9b8 to cfdab56 Compare February 3, 2026 17:51
@xieandrew xieandrew changed the title Rewrite delete logic to fix transformed partition types fix: Fix delete partition for transformed partition types Feb 6, 2026
@xieandrew xieandrew added the ice Relates to ice label Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ice Relates to ice

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ice delete partition only works if the hh:mm:dd values are passed along with the date.

1 participant