You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "shyjsarah (via GitHub)" <gi...@apache.org> on 2026/07/27 04:35:04 UTC
[PR] [python] Support pypaimon-rust 0.4 [paimon]
shyjsarah opened a new pull request, #8856:
URL: https://github.com/apache/paimon/pull/8856
## Purpose
`pypaimon[sql]` currently caps `pypaimon-rust` below 0.4.0. After paimon-rust bumped its workspace version to 0.4.0, installing the source-built Python binding followed by `pypaimon[sql]` makes pip reject the installed binding and try to resolve an older wheel.
## Changes
- Allow the SQL extra to use both pypaimon-rust 0.3.x and 0.4.x.
- Pin the Python 3.11 CI lane to the paimon-rust 0.4.0 version-bump commit.
- Install `pypaimon[sql]` in that lane so CI verifies the package metadata accepts the source-built binding.
## Testing
- Built and installed pypaimon-rust 0.4.0 from source in a clean Python 3.10 virtual environment.
- Verified `python -m pip install --dry-run './paimon-python[sql]'` recognizes the installed `pypaimon-rust 0.4.0`.
- Parsed `.github/workflows/paimon-python-checks.yml` as YAML.
- Ran `git diff --check`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [python] Support pypaimon-rust 0.4 [paimon]
Posted by "XiaoHongbo-Hope (via GitHub)" <gi...@apache.org>.
XiaoHongbo-Hope commented on code in PR #8856:
URL: https://github.com/apache/paimon/pull/8856#discussion_r3654905985
##########
paimon-python/setup.py:
##########
@@ -187,7 +187,7 @@ def read_requirements():
'paimon-ftindex==0.1.0; python_version>="3.8"',
],
'sql': [
- 'pypaimon-rust>=0.3.0,<0.4.0; python_version>="3.10"',
+ 'pypaimon-rust>=0.3.0,<0.5.0; python_version>="3.10"',
Review Comment:
pypaimon-rust 0.4 is built against datafusion-ffi 54, whose ABI is incompatible with
52/53 and future 55+. Could we pin datafusion to `datafusion>=54,<55` and add a real
PaimonCatalog registration test?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [python] Support pypaimon-rust 0.4 [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #8856:
URL: https://github.com/apache/paimon/pull/8856
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [python] Support pypaimon-rust 0.4 [paimon]
Posted by "shyjsarah (via GitHub)" <gi...@apache.org>.
shyjsarah commented on code in PR #8856:
URL: https://github.com/apache/paimon/pull/8856#discussion_r3656616869
##########
paimon-python/setup.py:
##########
@@ -187,7 +187,7 @@ def read_requirements():
'paimon-ftindex==0.1.0; python_version>="3.8"',
],
'sql': [
- 'pypaimon-rust>=0.3.0,<0.4.0; python_version>="3.10"',
+ 'pypaimon-rust>=0.3.0,<0.5.0; python_version>="3.10"',
Review Comment:
Updated in a15c0c4729: pinned DataFusion to >=54,<55 and replaced the hasattr check with an actual PaimonCatalog registration against DataFusion SessionContext.
##########
paimon-python/setup.py:
##########
@@ -187,7 +187,7 @@ def read_requirements():
'paimon-ftindex==0.1.0; python_version>="3.8"',
],
'sql': [
- 'pypaimon-rust>=0.3.0,<0.4.0; python_version>="3.10"',
+ 'pypaimon-rust>=0.3.0,<0.5.0; python_version>="3.10"',
Review Comment:
Done in a15c0c4729. Removed the pypaimon-rust upper bound and kept only >=0.3.0.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [python] Support pypaimon-rust 0.4 [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on code in PR #8856:
URL: https://github.com/apache/paimon/pull/8856#discussion_r3656422187
##########
paimon-python/setup.py:
##########
@@ -187,7 +187,7 @@ def read_requirements():
'paimon-ftindex==0.1.0; python_version>="3.8"',
],
'sql': [
- 'pypaimon-rust>=0.3.0,<0.4.0; python_version>="3.10"',
+ 'pypaimon-rust>=0.3.0,<0.5.0; python_version>="3.10"',
Review Comment:
Just remove `<`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [python] Support pypaimon-rust 0.4 [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #8856:
URL: https://github.com/apache/paimon/pull/8856#issuecomment-5104584485
+1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org