search

apache / iceberg-rust

Apache Iceberg
https://rust.iceberg.apache.org/
Apache License 2.0
675 stars 159 forks source link

fix: TableUpdate Snapshot deserialization for v1 #656

Closed c-thiel closed 1 month ago

c-thiel commented 1 month ago

The rest-spec specifies sequence-number as optional for TableUpdates: https://github.com/apache/iceberg/blob/09c737656d316ab6172e0d5ee6920869237e6fd1/open-api/rest-catalog-open-api.yaml#L2319-L2326

In the previous deserialization of updates using _serde::SnapshotV2 sequence-number is required.

This PR adds a separate deserialization function for catalogs that follows the REST spec.

I got errors from starrocks when writing V1 tables.