Skip to content

Conversation

@gabeiglio
Copy link
Contributor

To replicate this issue run the following spark queries:

CREATE TABLE ns.barfoo (dateint INT,hour INT, data STRING) USING iceberg PARTITIONED BY (dateint, hour);
INSERT INTO ns.barfoo (dateint, hour, data) VALUES (20250101, 1, 'a'), (20250102, 1, 'a'), (20250102, 2, 'a');
ALTER TABLE ns.barfoo DROP PARTITION FIELD dateint;
ALTER TABLE ns.barfoo DROP COLUMN dateint;
SELECT * FROM ns.barfoo.files;

@github-actions github-actions bot added the core label Sep 16, 2025
builder.add(newFieldId, newFieldId, field.name(), Transforms.identity());
}
return builder.build();
return builder.build(true);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a short comment to explain why we intentionally bypass the compatibility check here

Copy link
Contributor

@Fokko Fokko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with @huaxingao that a comment would be nice there, apart from that, LGTM! 👍

@gabeiglio gabeiglio force-pushed the fix-corrupted-metadata-table branch from 261f814 to 2a0341d Compare September 17, 2025 06:31
@gabeiglio
Copy link
Contributor Author

@huaxingao @Fokko thanks for the review! Added a comment 👍🏼

Copy link
Contributor

@singhpk234 singhpk234 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix @gabeiglio ! LGTM as well

@singhpk234 singhpk234 merged commit 87fce5f into apache:main Sep 17, 2025
42 checks passed
@singhpk234
Copy link
Contributor

Thank you for the fix @gabeiglio ! Thank you for the reviews @Fokko @huaxingao @bryanck !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants