First time writing rust, most of this code is generated but helped me understand the codebase.
Here are a couple of things I've learned

• bindings/python/src/lib.rs is how we can expose rust code as python binding
• The catalog implementation is in a different crate than the "catalog trait".
  • "catalog trait" is part of the "iceberg crate" https://github.com/apache/iceberg-rust/blob/main/crates/iceberg/src/catalog/mod.rs#L39
  • the SQL catalog implementation is part of the "catalog crate" https://github.com/apache/iceberg-rust/blob/main/crates/catalog/sql/src/catalog.rs#L71

Hi, @kevinjqliu I'm fine with adding it in pyiceberg_core module. But instead of adding sql catalog only, I think it's better to add a wrapper around Catalog trait? What do you think?

@liurenjie1024, following up on this comment #534 (comment)
Is the idea to expose only the Catalog trait to python? And perhaps a function to create different catalog implementations similar to what load_catalog is doing in python?

Is the idea to expose only the Catalog trait to python? And perhaps a function to create different catalog implementations similar to what load_catalog is doing in python?

@kevinjqliu Sorry for late reply. Yeah, this looks good to me.