Skip to content

Commit 904c0b7

Browse files
authored
Remove PyArrow version upper bound and update lockfile (#2262)
<!-- Thanks for opening a pull request! --> As title. Closes #2258. Updated the lockfile just in case but it didn't change the Arrow version. <!-- In the case this PR will resolve an issue, please replace ${GITHUB_ISSUE_ID} below with the actual Github issue id. --> <!-- Closes #${GITHUB_ISSUE_ID} --> # Rationale for this change PyIceberg doesn't need this upper bound since it uses stable public APIs of PyArrow (as far as I can tell). Dependency version upper bounds are known to be problematic: 1. Cause cascading dependency conflicts 2. Block security and bug fixes 3. Limit user flexibility In my case, this causes issues for Bodo's Arrow upgrades since both Arrow and PyIceberg are dependencies. # Are these changes tested? Existing tests. # Are there any user-facing changes? <!-- In the case of user-facing changes, please add the changelog label. --> None except allowing newer PyArrow versions.
1 parent 661d75a commit 904c0b7

File tree

3 files changed

+705
-605
lines changed

3 files changed

+705
-605
lines changed

dev/docker-compose-azurite.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,4 @@ services:
2222
hostname: azurite
2323
ports:
2424
- 10000:10000
25-
command: ["azurite-blob", "--loose", "--blobHost", "0.0.0.0"]
25+
command: ["azurite-blob", "--loose", "--skipApiVersionCheck", "--blobHost", "0.0.0.0"]

0 commit comments

Comments
 (0)