Skip to content

Correctly handle decimal physicial type mapping #1789

@Fokko

Description

@Fokko

Apache Iceberg version

None

Please describe the bug 🐞

According to the spec:
image

Here we map everything to a fixed byte array:

def visit_decimal(self, decimal_type: DecimalType) -> str:
return "FIXED_LEN_BYTE_ARRAY"

Let's add a test here with precision 8, 16 and 24:

Willingness to contribute

  • I can contribute a fix for this bug independently
  • I would be willing to contribute a fix for this bug with guidance from the Iceberg community
  • I cannot contribute a fix for this bug at this time

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions