This is an automated email from the ASF dual-hosted git repository.
kosiew pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion-python.git
The following commit(s) were added to refs/heads/main by this push:
new 5ce35f89 fix: clone parquet-testing into `parquet` to match submodule
path in verify script (#1617)
5ce35f89 is described below
commit 5ce35f8993151fe8cd8c4fb7865aee102752bc5b
Author: Adrian Garcia Badaracco <[email protected]>
AuthorDate: Fri Jul 10 01:11:05 2026 +0000
fix: clone parquet-testing into `parquet` to match submodule path in verify
script (#1617)
`verify-release-candidate.sh` clones the parquet-testing repository into a
`parquet-testing` directory, but the git submodule path declared in
`.gitmodules` is `parquet`, and every parquet-based test reads its data from
`parquet/data/...` (e.g. `python/tests/test_io.py`, `test_store.py`,
`test_context.py`). The arrow-testing clone one line above already correctly
uses the submodule path (`testing`).
This is currently latent because the script's `python3 -m pytest`
invocation is
commented out, so the wrong directory is never exercised. Fixing the path
is a
prerequisite for enabling the test run during release verification.
Claude-Session: https://claude.ai/code/session_01Pj5DVU7MaammM2nfHh1ZRG
Co-authored-by: Claude Opus 4.8 (1M context) <[email protected]>
---
dev/release/verify-release-candidate.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dev/release/verify-release-candidate.sh
b/dev/release/verify-release-candidate.sh
index 503e0de6..42e3970f 100755
--- a/dev/release/verify-release-candidate.sh
+++ b/dev/release/verify-release-candidate.sh
@@ -132,7 +132,7 @@ test_source_distribution() {
# Clone testing repositories into the expected location
git clone https://github.com/apache/arrow-testing.git testing
- git clone https://github.com/apache/parquet-testing.git parquet-testing
+ git clone https://github.com/apache/parquet-testing.git parquet
python3 -m venv .venv
if [ -x ".venv/bin/python" ]; then
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
