Skip to content

Commit 25114c5

Browse files
authored
[55_maintenance] Fix verify-release-candidate.sh by skipping arrow-pyarrow and parquet-variant (#7745)
Note this targets a release branch, not main I have a different proposed fix for `main`: - #7742 I will also make a fix for parquet-variant test failures # Which issue does this PR close? - Related to #7394 - Related to #7736 - Related to #7746 # Rationale for this change `cargo test --all` requires python and pyarrow installed, where it did not in previous versions of arrow due to breaking out `arrow-pyarrow` into its own crate in - #7694 Also the new `parquet-variant` crate tests fail as part of the verification script too -- see ttps://github.com//issues/7746 In order to get a script that can automatically verify a release candidate, let's ignore this new module for now. More details - #7736 - #7742 Note that the arrow-pyarrow tests do run as part of CI and succeed on this branch # What changes are included in this PR? 1. Exclude running the `arrow-pyarrow` and `parquet-variant` tests as part of `verify-release-acndidate` # Testing I verified locally that `./dev/release/verify-release-candidate.sh 55.2.0 1` passes with this script # Are there any user-facing changes? No this is a development process only change
1 parent df702cf commit 25114c5

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

dev/release/verify-release-candidate.sh

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,11 @@ test_source_distribution() {
113113
export ARROW_TEST_DATA=$PWD/arrow-testing-data/data
114114
export PARQUET_TEST_DATA=$PWD/parquet-testing-data/data
115115

116-
cargo test --all
116+
# ignore arrow-pyarrow due to
117+
# https://github.com/apache/arrow-rs/issues/7736
118+
# ignore parquet-variant due to
119+
# https://github.com/apache/arrow-rs/issues/7746
120+
cargo test --all --exclude arrow-pyarrow --exclude parquet-variant
117121

118122
# verify that the leaf crates can be published to crates.io
119123
# we can't verify crates that depend on others

0 commit comments

Comments
 (0)