kazdy commented on code in PR #149:
URL: https://github.com/apache/hudi-rs/pull/149#discussion_r1770519805
##########
python/pyproject.toml:
##########
@@ -23,19 +23,18 @@ build-backend = "maturin"
name = "hudi"
description = "Native Python binding for Apache Hudi, based on hudi-rs."
urls = { repository = "https://github.com/apache/hudi-rs/tree/main/python/"; }
-requires-python = ">=3.8"
+requires-python = ">=3.9"
keywords = ["apachehudi", "hudi", "datalake", "arrow"]
license = "Apache License 2.0"
classifiers = [
"License :: OSI Approved :: Apache Software License",
- "Programming Language :: Python :: 3.8",
"Programming Language :: Python :: 3.9",
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12"
]
dependencies = [
- "pyarrow>=8",
+ "pyarrow>=17",
"pyarrow-hotfix",
Review Comment:
pyarrow-hotfix is not needed since 14.0.1
I don't have a strong opinion on upgrading this, it seemed like a good idea,
but I can revert this.
edit:
I checked some popular libraries that use pyarrow and declare a dependency:
datafusion: pyarrow>=11.0.0,
polars: pyarrow >= 7.0.0,
daft: pyarrow >= 7.0.0,
lance: pyarrow>=12
so should not cause issues for users (?).
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
