-
Notifications
You must be signed in to change notification settings - Fork 124
Conversation
Add `kNimble` to the `FileFormatType` enum along with its `ToString` and `FileFormatTypeFromString` mappings, so tables whose data files are written in the Nimble columnar format can be represented and round-tripped through manifest and REST JSON deserialization.
wgtmac
commented
Aug 8, 2026
IMHO, we cannot make this change until nimble has been formally accepted by the Iceberg spec. It would be good to start a discussion on dev@iceberg.apache.org to see what's the requirement to add a new file format. Vortex is doing this: https://docs.google.com/document/d/1g0EJDr3DkSmfAce8C94zA2JyWRj2SeNeAeiYOotnSww
zzhao0
commented
Aug 10, 2026
Thanks for pointing me to the direction. Let me read through this.
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.
Working on integrating Nimble https://github.com/facebookincubator/nimble to iceberg. This requires extending the FileFormatType enum.
Add
kNimbleto theFileFormatTypeenum along with itsToStringandFileFormatTypeFromStringmappings, so tables whose data files are written in the Nimble columnar format can be represented and round-tripped through manifest and REST JSON deserialization.