Update verify_release.sh to work with 30.0.0 via workaround#3413
Conversation
tustvold
left a comment
There was a problem hiding this comment.
I'm not a fan of this, but if it unblocks things we can fix it properly later
|
I tested that this script can successfully verify 30.0.0 rc 1 |
|
Filed #3414 to track the underlying issue |
|
Benchmark runs are scheduled for baseline = 9b57621 and contender = 9398af6. 9398af6 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Which issue does this PR close?
closes #3410
Rationale for this change
parquet now has an optional dependence on
object_storebut object_store is not part of the tarball and thus the verify script failsWhat changes are included in this PR?
A hack to the
verify_releasescript to fix the verification and remove thepathmarker in the object store dependency declaration@tustvold has noted that rather than changing the verify script the actual tarball should probably be fixed (aka so that it does not have references to
path. I will write up a follow on ticket for this issueAre there any user-facing changes?
No