Skip to content

Commit 5f05093

Browse files
committed
ci: re-enable datafusion 53 test lane
Signed-off-by: Ethan Urbanski <ethan@urbanskitech.com>
1 parent 376a5ad commit 5f05093

2 files changed

Lines changed: 2 additions & 28 deletions

File tree

.github/workflows/python_build.yml

Lines changed: 0 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,6 @@ jobs:
7171
test-datafusion:
7272
name: Python Build (Python 3.10 DataFusion integration)
7373
runs-on: ubuntu-latest
74-
continue-on-error: true
7574

7675
steps:
7776
# v6.0.2
@@ -84,35 +83,10 @@ jobs:
8483
- name: Build and install deltalake
8584
run: make develop
8685

87-
- name: Install DataFusion when available
88-
id: datafusion
89-
shell: bash
90-
run: |
91-
set +e
92-
output="$(uv pip install 'datafusion>=53,<54' 2>&1)"
93-
status=$?
94-
set -e
95-
printf '%s\n' "$output"
96-
97-
if [ "$status" -eq 0 ]; then
98-
echo "available=true" >> "$GITHUB_OUTPUT"
99-
exit 0
100-
fi
101-
102-
if printf '%s\n' "$output" | grep -Eq "No matching distribution|No solution found|not found in the package registry"; then
103-
echo "available=false" >> "$GITHUB_OUTPUT"
104-
echo "::notice::Skipping Python DataFusion integration tests because no compatible DataFusion 53 distribution is available."
105-
exit 0
106-
fi
107-
108-
exit "$status"
109-
11086
- name: Assert DataFusion major version
111-
if: steps.datafusion.outputs.available == 'true'
11287
run: uv run --no-sync python -c "from importlib.metadata import version; v=version('datafusion'); assert v.split('.')[0]=='53', f'Expected 53.x, got {v}'"
11388

11489
- name: Run DataFusion integration test
115-
if: steps.datafusion.outputs.available == 'true'
11690
run: DELTALAKE_RUN_DATAFUSION_TESTS=1 uv run --no-sync pytest -v tests/test_datafusion.py -m datafusion
11791

11892
test-lakefs:

python/pyproject.toml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ dev = [
111111
]
112112
opentelemetry = ["opentelemetry-sdk>=1.20.0", "opentelemetry-api>=1.20.0"]
113113
polars = ["polars==1.32"]
114+
datafusion = ["datafusion==53.*"]
114115
lakefs = ["lakefs==0.8.0"]
115116
pyspark = [
116117
"pyspark==4.0.1",
@@ -127,5 +128,4 @@ other = [
127128
]
128129

129130
[tool.uv]
130-
# Re-enable a Python DataFusion dependency group once DataFusion 53 wheels are published.
131-
default-groups = ["dev", "opentelemetry", "polars", "lakefs", "docs", "other"]
131+
default-groups = ["dev", "opentelemetry", "polars", "datafusion", "lakefs", "docs", "other"]

0 commit comments

Comments
 (0)