You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "blackmwk (via GitHub)" <gi...@apache.org> on 2026/05/25 13:09:08 UTC

[D] Add bindings/python into root workspace. [iceberg-rust]

GitHub user blackmwk created a discussion: Add bindings/python into root workspace.

Currently the python binding, e.g pyiceberg_core is in a standalone workspace, and excludes from root workspace. This cause some troubles for us:
1. Missing check for pyiceberg_core. In root workspace, we have several jobs to check things like toml, clippy, rust format. But these are missing for `pyiceberg_core` crate.
2. Diverged dependency version management. The root workspace shares a lot dependencies with with `pyiceberg_core`, such as `tokio`, `datafusion`, etc. Due to the possible instability of public apis such as arrow and datafusion, this may cause trouble for us to maintain.

We are in fact releasing `pyiceberg_core` with core iceberg crates with same schedule, so I would suggest to move `pyiceberg_core` to root workspace.

GitHub link: https://github.com/apache/iceberg-rust/discussions/2505

----
This is an automatically sent email for issues@iceberg.apache.org.
To unsubscribe, please send an email to: issues-unsubscribe@iceberg.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [D] Add bindings/python into root workspace. [iceberg-rust]

Posted by "blackmwk (via GitHub)" <gi...@apache.org>.
GitHub user blackmwk closed a discussion: Add bindings/python into root workspace.

Currently the python binding, e.g pyiceberg_core is in a standalone workspace, and excludes from root workspace. This cause some troubles for us:
1. Missing check for pyiceberg_core. In root workspace, we have several jobs to check things like toml, clippy, rust format. But these are missing for `pyiceberg_core` crate.
2. Diverged dependency version management. The root workspace shares a lot dependencies with with `pyiceberg_core`, such as `tokio`, `datafusion`, etc. Due to the possible instability of public apis such as arrow and datafusion, this may cause trouble for us to maintain.

We are in fact releasing `pyiceberg_core` with core iceberg crates with same schedule, so I would suggest to move `pyiceberg_core` to root workspace.

GitHub link: https://github.com/apache/iceberg-rust/discussions/2505

----
This is an automatically sent email for issues@iceberg.apache.org.
To unsubscribe, please send an email to: issues-unsubscribe@iceberg.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [D] Add bindings/python into root workspace. [iceberg-rust]

Posted by "Fokko (via GitHub)" <gi...@apache.org>.
GitHub user Fokko added a comment to the discussion: Add bindings/python into root workspace.

This makes sense to me, I don't think we want to release `pyiceberg_core` seperately, so that's not an issue 👍 

GitHub link: https://github.com/apache/iceberg-rust/discussions/2505#discussioncomment-17050505

----
This is an automatically sent email for issues@iceberg.apache.org.
To unsubscribe, please send an email to: issues-unsubscribe@iceberg.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [D] Add bindings/python into root workspace. [iceberg-rust]

Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
GitHub user kevinjqliu added a comment to the discussion: Add bindings/python into root workspace.

+1 great idea. we dont need to maintain a completely separate crates/package under `bindings/python` 

GitHub link: https://github.com/apache/iceberg-rust/discussions/2505#discussioncomment-17051925

----
This is an automatically sent email for issues@iceberg.apache.org.
To unsubscribe, please send an email to: issues-unsubscribe@iceberg.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


Re: [D] Add bindings/python into root workspace. [iceberg-rust]

Posted by "blackmwk (via GitHub)" <gi...@apache.org>.
GitHub user blackmwk closed the discussion with a comment: Add bindings/python into root workspace.

Seems we reached consensus on it, resolved in #2506 , thanks @kevinjqliu !

GitHub link: https://github.com/apache/iceberg-rust/discussions/2505#discussioncomment-17087408

----
This is an automatically sent email for issues@iceberg.apache.org.
To unsubscribe, please send an email to: issues-unsubscribe@iceberg.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org