fix: Sort Order ID in TableMetadataBuilder changes should be updated (#909)

c-thiel · web-flow · commit 2688e50fdc24 · 2025-01-23T23:20:01.000+08:00
diff --git a/crates/iceberg/src/spec/table_metadata_builder.rs b/crates/iceberg/src/spec/table_metadata_builder.rs
@@ -831,7 +831,7 @@ impl TableMetadataBuilder {
         if sort_order_found {
             if self.last_added_order_id != Some(new_order_id) {
                 self.changes.push(TableUpdate::AddSortOrder {
-                    sort_order: sort_order.clone(),
+                    sort_order: sort_order.clone().with_order_id(new_order_id),
                 });
                 self.last_added_order_id = Some(new_order_id);
             }

Original file line number	Diff line number	Diff line change
`@@ -831,7 +831,7 @@ impl TableMetadataBuilder {`
`831`	`831`	`if sort_order_found {`
`832`	`832`	`if self.last_added_order_id != Some(new_order_id) {`
`833`	`833`	`self.changes.push(TableUpdate::AddSortOrder {`
`834`		`- sort_order: sort_order.clone(),`
	`834`	`+ sort_order: sort_order.clone().with_order_id(new_order_id),`
`835`	`835`	`});`
`836`	`836`	`self.last_added_order_id = Some(new_order_id);`
`837`	`837`	`}`