feat: expose catalog metadata api for pypaimon_rust#306
Conversation
leaves12138
left a comment
There was a problem hiding this comment.
Thanks for the PR. The runtime binding implementation looks generally good, and the existing CI is green. I also verified locally that cargo test -q -p pypaimon_rust --no-default-features passes after the first build.
One blocking issue: this package ships py.typed and bindings/python/python/pypaimon_rust/datafusion.pyi, but the stub file still only exposes PaimonCatalog.__init__ and __datafusion_catalog_provider__. The new public APIs added by this PR (list_databases, list_tables, get_table, plus the new Table, TableSchema, and DataField classes and their methods) are missing from the type stubs.
That means dynamic Python calls work, but type-aware users and IDEs will still see the old API. Please update bindings/python/python/pypaimon_rust/datafusion.pyi to match the new exported Python surface. It would also be helpful to cover this with a small stub/import check if the project already has a place for that.
Also, GitHub currently reports the PR as not mergeable (mergeable_state=dirty), so it will need a rebase/update before merge.
|
@leaves12138 Thanks for the comment. I've updated and rebbased the branch. |
Purpose
Linked issue: close #284
Brief change log
Tests
API and Format
Documentation