This is an automated email from the ASF dual-hosted git repository.
liurenjie1024 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-rust.git
The following commit(s) were added to refs/heads/main by this push:
new 30ced991 feat: TableIdent derives PartialOrd (#1670)
30ced991 is described below
commit 30ced991997425e721ff1e1f1dcde0c49b0ec485
Author: Jannik Steinmann <[email protected]>
AuthorDate: Mon Sep 22 03:31:36 2025 +0200
feat: TableIdent derives PartialOrd (#1670)
## 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 <[email protected]>
---
crates/iceberg/src/catalog/mod.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/crates/iceberg/src/catalog/mod.rs
b/crates/iceberg/src/catalog/mod.rs
index f9e17808..77b171ee 100644
--- 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,
