feat: TableIdent derives PartialOrd (#1670)

DerGut · Fokko · web-flow · commit 30ced9919974 · 2025-09-22T09:31:36.000+08:00
## Which issue does this PR close? Closes #1669 ## What changes are included in this PR? `TableIdent` now derives `PartialOrd`. ## Are these changes tested? No, it's only derived behavior. I'm happy to add a test, but I'm not 100% convinced it would add value. --------- Co-authored-by: Fokko Driesprong <fokko@apache.org>
diff --git a/crates/iceberg/src/catalog/mod.rs b/crates/iceberg/src/catalog/mod.rs
@@ -225,7 +225,7 @@ impl Display for NamespaceIdent {
 }
 
 /// TableIdent represents the identifier of a table in the catalog.
-#[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, Hash)]
+#[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub struct TableIdent {
     /// Namespace of the table.
     pub namespace: NamespaceIdent,

Original file line number	Diff line number	Diff line change
`@@ -225,7 +225,7 @@ impl Display for NamespaceIdent {`
`225`	`225`	`}`
`226`	`226`
`227`	`227`	`/// TableIdent represents the identifier of a table in the catalog.`
`228`		`-#[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, Hash)]`
	`228`	`+#[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]`
`229`	`229`	`pub struct TableIdent {`
`230`	`230`	`/// Namespace of the table.`
`231`	`231`	`pub namespace: NamespaceIdent,`