Skip to content

Incorrect type definition for identifier-field-ids #1487

Open
@nt-tryg

Description

@nt-tryg

Apache Iceberg Rust version

None

Describe the bug

According to the iceberg spec documentation the field "identifier-field-ids" is an optional field.
See: https://iceberg.apache.org/spec/#schemas

But the type definition for it in rust is not marked as an Option.
See: https://github.com/apache/iceberg-rust/blob/main/crates/iceberg/src/spec/schema/mod.rs#L61C27-L61C40

Should be an easy change to comply with the official spec documentation, I can prepare a merge request for this.

To Reproduce

Expected behavior

Should match official iceberg spec

Willingness to contribute

I can contribute a fix for this bug independently

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions